-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Support monorepos without synchronized versions #45
Comments
Here are my thoughts on what sort of changes we could make to this action in order to achieve this goal. DisclaimerIt's important to know up front that right now this action reads the version we want to release (the "release version") from the branch being merged. This is being changed in #46 to read from the version in Current flowOnce we know what the release version is, we follow one of two workflows: Polyrepo
Monorepo
New flowFor polyrepos, everything should work how it works now. But for monorepos, there's a problem with the existing flow: it assumes that when a release is created, all packages will be bumped to the same version. This may continue to be true for Therefore, the changes we need to make are as follows:
Decisions to makeIn summary, here are the things we need to decide, in the case of a monorepo with an independent versioning strategy:
Prior artFor inspiration, take a look at Vercel, which more or less follows the same process that we want to follow: https://github.com/vercel/vercel |
Just an update on this: Instead of having this action take a second input, we should read the value of |
Another update: instead of using a version format of |
Here's the relevant part of the Notion doc which describes how this action fits into the larger workflow. This might be helpful to gain more context: https://www.notion.so/Core-Monorepo-Release-Automation-a65b28ab89a14c9d82dcf9fabc3eb1e1#aa00c0fc1e404448884f41c4998602e6 |
Currently this action only supports monorepos with synchronized versions. We will soon be using monorepos that don't use synchronized versions, so this will need to be updated to support that.
The text was updated successfully, but these errors were encountered: