Skip to content

Release 74.0.0#1634

Merged
mcmire merged 12 commits intomainfrom
release/74.0.0
Aug 28, 2023
Merged

Release 74.0.0#1634
mcmire merged 12 commits intomainfrom
release/74.0.0

Conversation

@mcmire
Copy link
Copy Markdown
Contributor

@mcmire mcmire commented Aug 25, 2023

This release is somewhat special. Most of these packages are just bumping dependencies on other packages within the monorepo. We recently made a change which ensures that if the version of a package A is bumped, then the new version of A is reflected in the manifest of packages that depend on A. This doesn't necessarily mean that those packages need to be released right away — it's fine if just A is released — but we are releasing every package right now to reduce the amount of changes that need to be published in the next release.

@mcmire mcmire requested review from a team as code owners August 25, 2023 17:55
Comment thread packages/assets-controllers/CHANGELOG.md Outdated
Comment thread packages/transaction-controller/CHANGELOG.md Outdated
Comment thread packages/preferences-controller/CHANGELOG.md

## [3.2.1]
### Changed
- There are no consumer-facing changes to this package. This version is a part of a synchronized release across all packages in our monorepo.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we do this for controller-utils as well? It's the only package not included in this release at the moment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

To consider the contrary: Is it actually worth it to publish a no-op release - could we skip base-controller (and any dependent which by this also becomes identical to previous version and didn't have any other dep updates since its last release) for this release? Not sure I understand the upside here.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The create-release-branch tool we use to create releases highlights every package with changes when you go to create a new release, with the expectation that the author of the release PR would review each of these packages to determine whether they need to be released (and if so, which version they should have).

This release is intended to clear the slate, so that future releases are easier to manage.

This situation will be less of a problem soon (we have some improvements to create-release-branch that we'll be working on soon to make it a bit smarter about releasing one package without the others safely). But that's a week or two out at best.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That's true. We also changed how packages refer to other packages within the monorepo. Now they use static versions always instead of dynamic versions which get resolved at publish-time. So we also want to publish every package regardless of whether there were actual changes to ensure that all packages have up-to-date references (and expose any issues that we may see as a result of this change sooner rather than later).

@mcmire
Copy link
Copy Markdown
Contributor Author

mcmire commented Aug 28, 2023

Ready for another review.

Gudahtt
Gudahtt previously approved these changes Aug 28, 2023
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@mcmire mcmire merged commit 3a7c474 into main Aug 28, 2023
@mcmire mcmire deleted the release/74.0.0 branch August 28, 2023 18:53
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants