Skip to content

Conversation

@coratgerl
Copy link
Contributor

@coratgerl coratgerl commented Nov 23, 2025

Pull Request

Remove unused dependencies or move to depDependencies

Summary by CodeRabbit

  • Chores
    • Updated project dependencies for improved development workflow and optimized bundle size.
    • Reorganized build tooling dependencies.
    • Removed unused dependencies.

✏️ Tip: You can customize this high-level summary in your review settings.

@parse-github-assistant
Copy link

I will reformat the title to use the proper commit message syntax.

@parse-github-assistant parse-github-assistant bot changed the title fix: remove unused dependencies fix: Remove unused dependencies Nov 23, 2025
@parse-github-assistant
Copy link

parse-github-assistant bot commented Nov 23, 2025

🚀 Thanks for opening this pull request!

@coderabbitai
Copy link

coderabbitai bot commented Nov 23, 2025

📝 Walkthrough

Walkthrough

Reorganized package dependencies by moving @babel/eslint-parser from dependencies to devDependencies, and removing graphql-tag, router, flow-bin, and husky packages entirely.

Changes

Cohort / File(s) Summary
Dependency reorganization
package.json
Moved @babel/eslint-parser from dependencies to devDependencies; removed graphql-tag and router from dependencies; removed flow-bin and husky from devDependencies

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

  • Verify that @babel/eslint-parser is appropriate as a devDependency (linting tool)
  • Confirm that graphql-tag and router are not used in the codebase or have been replaced
  • Ensure flow-bin and husky removals don't break existing development workflows

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete. It lacks the Issue section with a link/reference, detailed Approach description, and Tasks checklist required by the template. Complete the description by filling in the 'Closes' field with an issue reference, providing a detailed Approach section explaining the rationale for dependency changes, and checking applicable Tasks.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title 'perf: Remove unused dependencies' accurately describes the main change of removing and reorganizing dependencies, though the conventional commit type 'perf' may be slightly imprecise for a dependency restructuring task.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 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.

@parseplatformorg
Copy link
Contributor

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.

@codecov
Copy link

codecov bot commented Nov 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.08%. Comparing base (12beb8f) to head (6dbe25e).
⚠️ Report is 3 commits behind head on alpha.

Additional details and impacted files
@@            Coverage Diff             @@
##            alpha    #9943      +/-   ##
==========================================
+ Coverage   93.07%   93.08%   +0.01%     
==========================================
  Files         188      188              
  Lines       15298    15298              
  Branches      177      177              
==========================================
+ Hits        14238    14240       +2     
+ Misses       1048     1046       -2     
  Partials       12       12              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@mtrezza mtrezza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find!

@mtrezza mtrezza changed the title fix: Remove unused dependencies refactor: Remove unused dependencies Nov 23, 2025
@mtrezza mtrezza changed the title refactor: Remove unused dependencies perf: Remove unused dependencies Nov 23, 2025
@mtrezza mtrezza merged commit d4c6de0 into parse-community:alpha Nov 23, 2025
27 of 28 checks passed
parseplatformorg pushed a commit that referenced this pull request Nov 23, 2025
# [8.5.0-alpha.15](8.5.0-alpha.14...8.5.0-alpha.15) (2025-11-23)

### Performance Improvements

* Remove unused dependencies ([#9943](#9943)) ([d4c6de0](d4c6de0))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 8.5.0-alpha.15

@parseplatformorg parseplatformorg added the state:released-alpha Released as alpha version label Nov 23, 2025
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.

3 participants