-
Notifications
You must be signed in to change notification settings - Fork 14
style: Format all files except autogenerated ones #307
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR applies Prettier formatting across the codebase (excluding autogenerated files). The changes are purely stylistic and do not alter any business logic or functionality.
Key changes:
- Reorganized import statements to follow a consistent order
- Reformatted code blocks, arrays, and line breaks for consistency
- Adjusted spacing and indentation in markdown files
Reviewed Changes
Copilot reviewed 19 out of 22 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| utils/proxy-tools.ts | Reordered imports alphabetically |
| scripts/upgrades/v6.1.0-bulk-processing.ts | Reordered imports alphabetically |
| scripts/upgrades/v6.1.0-bulk-processing.md | Consolidated multi-line list into single line |
| scripts/upgrades/README.md | Added blank lines before code blocks for consistency |
| scripts/tools/sol-to-uml.mjs | Reformatted array literals and function calls, added semicolons |
| scripts/tools/puml-to-links.mjs | Added semicolons for consistency |
| scripts/sponsoring/upgrade-status.md | Removed extra blank lines |
| scripts/sponsoring/README.md | Added blank lines before code blocks for consistency |
| scripts/sponsoring/1_add-modules-to-proxy.ts | Reordered imports alphabetically |
| scripts/boost/README.md | Added blank lines before code blocks for consistency |
| scripts/boost/1_add-modules-to-proxy.ts | Reordered imports alphabetically |
| docs/diagrams.md | Added blank lines for section spacing |
| docs/README.md | Added blank lines before code blocks, removed extra blank lines |
| contracts/facets/IexecPoco1Facet.sol | Reformatted multi-line inheritance declaration |
| contracts/IexecInterfaceToken.sol | Moved comment above interface declaration |
| contracts/IexecInterfaceNative.sol | Moved comment above interface declaration |
| README.md | Added blank line for consistency |
| CHANGELOG.md | Added blank lines for section spacing, reformatted nested lists |
| .prettierrc | Consolidated plugins array into single line |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #307 +/- ##
=======================================
Coverage 84.85% 84.85%
=======================================
Files 37 37
Lines 1241 1241
Branches 235 235
=======================================
Hits 1053 1053
Misses 188 188 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 20 out of 23 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 19 out of 23 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 20 out of 24 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
npx prettier --write .