-
Notifications
You must be signed in to change notification settings - Fork 1
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
chore: Align Change Sets for 1.0.0 release #164
chore: Align Change Sets for 1.0.0 release #164
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files
☔ View full report in Codecov by Sentry. |
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found.
📢 Thoughts on this report? Let us know! |
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.
The changes denote an important transition of a set of plugins to major versions, indicating some significant updates. It's also noticeable that the mode is being switched from 'pre' to 'exit'. However, I couldn't find any changes related to the version bump in the presented diff. It would be helpful to review these changes in addition to the alterations in the mode and version declarations.
@@ -1,5 +1,5 @@ | |||
{ | |||
"mode": "pre", | |||
"mode": "exit", |
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.
You have changed the mode from 'pre' to 'exit'. It's hard to determine the consequences of this change without knowing how this mode is used. Ensure that other parts of your application are able to properly handle this updated mode.
@@ -0,0 +1,13 @@ | |||
--- | |||
"@codecov/bundler-plugin-core": major |
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.
You have listed multiple packages as 'major' in the changeset metadata. If this is accurately reflecting the changes that have been made to these packages, that's fine. However, make sure that you are following semantic versioning guidelines and that these updates truly warrant a major version increment.
.changeset/shiny-ghosts-play.md
Outdated
"@codecov/webpack-plugin": major | ||
--- | ||
|
||
Release 1.0.0 of the plugins |
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.
This change summary isn't descriptive enough about what exactly has been modified in this new release. While it's clear that it's a considerable release of the plugins with version 1.0.0, it would be beneficial to summarize briefly what new features or major changes are included in this release.
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.
I assume this is what will show up in https://github.com/codecov/codecov-javascript-bundler-plugins/releases, so let's add those extra docs as the bot says.
- Link to codecov marketing/docs
- Talk about what it supports atm
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.
I'll update this, but I may have gone a bit too far in MD formatting. So I'll run a pre-publish to see what the result looks like before merging.
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.
Bundle ReportChanges will increase total bundle size by 445.36kB ⬆️
|
Bundle ReportChanges will decrease total bundle size by 29.16kB (-0.51%) ⬇️. This is within the configured threshold ✅ Detailed changes
|
.changeset/shiny-ghosts-play.md
Outdated
"@codecov/webpack-plugin": major | ||
--- | ||
|
||
Release 1.0.0 of the plugins |
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.
I assume this is what will show up in https://github.com/codecov/codecov-javascript-bundler-plugins/releases, so let's add those extra docs as the bot says.
- Link to codecov marketing/docs
- Talk about what it supports atm
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.
CodecovAI submitted a new review for 54ea139
.changeset/shiny-ghosts-play.md
Outdated
"@codecov/webpack-plugin": major | ||
--- | ||
|
||
Release 1.0.0 of the plugins. You can read more about the plugins in our [docs](https://docs.codecov.com/docs/javascript-bundle-analysis). |
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.
Ensure that all these plugins are well-documented and each major change is described sufficiently in the documentation pointed at by this link.
|
||
The plugins have the following functionality: | ||
|
||
- Automatically collect and upload bundle stats data to Codecov |
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.
This functionality seems beneficial, but make sure that implementing this feature doesn't violate any user's privacy or security principles.
.changeset/shiny-ghosts-play.md
Outdated
|
||
- Automatically collect and upload bundle stats data to Codecov | ||
- Tokenless uploads of bundle stats for forked upstream pull requests | ||
- GH OIDC authentication for users or organizations who have configured it with GitHub |
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.
Ensure the authentication flows are vigorously tested and secure from potential breaches.
Description
This PR sets everything up for the
1.0.0
release of the plugins, adding in a major release as well as exiting pre-mode. Here is an example after runningprepare-publish
ensuring that everything is working as expected with changeset: #165