Skip to content

Conversation

@Olexandr88
Copy link
Contributor

@Olexandr88 Olexandr88 commented Jan 17, 2026

Summary by CodeRabbit

  • Chores
    • Removed a development formatting plugin from dev dependencies.
    • No functional or user-facing changes; runtime behavior and error handling are unaffected.

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 17, 2026

Walkthrough

The pull request removes the devDependency prettier-plugin-solidity from package.json. No other dependencies, configuration, functionality, or control flow changes are present.

Changes

Cohort / File(s) Summary
Dependency Removal
package.json
Removed devDependency entry for prettier-plugin-solidity (one-line deletion).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning No pull request description was provided by the author, but the description template only requires a 'Description of the changes' section. Add a pull request description explaining why prettier-plugin-solidity was removed and confirm it is no longer needed.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: removing an unused prettier-plugin-solidity dependency.
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

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!


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.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 17, 2026

Greptile Summary

Removed the unused prettier-plugin-solidity package from devDependencies. The project uses solhint for Solidity linting rather than Prettier, and no references to this plugin exist in the codebase or configuration files. This cleanup reduces dependency bloat without affecting functionality.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The change removes an unused dependency with no references in the codebase. The project uses solhint for Solidity formatting, making this plugin unnecessary. No configuration files reference it, and all transitive dependencies are properly cleaned from yarn.lock
  • No files require special attention

Important Files Changed

Filename Overview
package.json Removed unused prettier-plugin-solidity dependency from devDependencies
yarn.lock Removed lockfile entries for prettier-plugin-solidity and its transitive dependencies

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant PM as Package Manager
    participant Repo as Repository
    
    Dev->>Repo: Remove prettier-plugin-solidity from package.json
    Dev->>PM: Run yarn install
    PM->>PM: Remove plugin dependencies from yarn.lock
    PM->>PM: Clean up transitive dependencies
    Note over PM: Removes @solidity-parser/parser@^0.14.0
    Note over PM: Removes emoji-regex@^10.0.0
    Note over PM: Removes solidity-comments-extractor@^0.0.7
    PM->>Repo: Update yarn.lock with cleaned dependencies
    Note over Repo: Dependency tree simplified
    Note over Repo: Solidity formatting still handled by solhint
Loading

@MantisClone MantisClone self-requested a review January 30, 2026 16:44
@MantisClone MantisClone merged commit 4aba9aa into RequestNetwork:master Jan 30, 2026
1 of 3 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in Request Network Tech Backlog Jan 30, 2026
@MantisClone
Copy link
Member

Congratulations, your pull request has been merged! Thank you for your valuable contribution to Request Network. As a reminder, every merged PR is automatically entered into our Best PR Initiative, offering a quarterly prize of $500. Your work significantly supports our project's growth, and we encourage you to continue engaging with our community. Additionally, if you want to build or add crypto payments and invoicing features, explore how our API can reduce deployment time from months to hours while offering advanced features. Get in touch to learn more and fast-track your development.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants