Skip to content

fix: Remove unused dependencies#3227

Merged
mtrezza merged 2 commits intoparse-community:alphafrom
mtrezza:fix/unused-imports
Feb 19, 2026
Merged

fix: Remove unused dependencies#3227
mtrezza merged 2 commits intoparse-community:alphafrom
mtrezza:fix/unused-imports

Conversation

@mtrezza
Copy link
Copy Markdown
Member

@mtrezza mtrezza commented Feb 19, 2026

Pull Request

Issue

Remove unused dependencies.

Summary by CodeRabbit

  • Chores
    • Removed unused development dependencies to streamline the project setup.

@parse-github-assistant
Copy link
Copy Markdown

parse-github-assistant bot commented Feb 19, 2026

🚀 Thanks for opening this pull request!

@mtrezza mtrezza changed the title fix: Remove unused dependency imports fix: Remove unused dependencies Feb 19, 2026
@parseplatformorg
Copy link
Copy Markdown
Contributor

parseplatformorg commented Feb 19, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 19, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

Two devDependencies, "request" and "request-promise", were removed from package.json. All other dependencies and scripts remain unchanged. This is a straightforward dependency cleanup with no impact to runtime dependencies.

Changes

Cohort / File(s) Summary
Dependency Cleanup
package.json
Removed request and request-promise from devDependencies; no changes to runtime dependencies or scripts.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description is incomplete. It includes the Issue section but is missing the required Approach and Tasks sections from the template. Add the Approach section describing the specific changes made (removed 'request' and 'request-promise' devDependencies) and complete the Tasks section.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: removing unused dependencies from package.json.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mtrezza mtrezza merged commit 3ba250d into parse-community:alpha Feb 19, 2026
11 checks passed
@mtrezza mtrezza deleted the fix/unused-imports branch February 19, 2026 23:36
parseplatformorg pushed a commit that referenced this pull request Feb 19, 2026
## [9.0.1-alpha.2](9.0.1-alpha.1...9.0.1-alpha.2) (2026-02-19)

### Bug Fixes

* Remove unused dependencies ([#3227](#3227)) ([3ba250d](3ba250d))
@parseplatformorg
Copy link
Copy Markdown
Contributor

🎉 This change has been released in version 9.0.1-alpha.2

@parseplatformorg parseplatformorg added the state:released-alpha Released as alpha version label Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state:released-alpha Released as alpha version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants