Skip to content

Reformat root version to include the release no. #22

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

Merged
merged 6 commits into from
Sep 22, 2022

Conversation

mcmire
Copy link
Contributor

@mcmire mcmire commented Jul 22, 2022

This commit changes how the root version is bumped per release so that
instead of looking like <yyyy>.<mm>.<dd>, it looks like
<yyyy><mm><dd>.<release-number>.0. That is, the root version
consists of the date that the release was created + a release number,
which is incremented every time a new release is issued.


Fixes #16.

@mcmire mcmire requested a review from a team as a code owner July 22, 2022 23:31
@mcmire mcmire self-assigned this Jul 25, 2022
@mcmire mcmire marked this pull request as draft July 29, 2022 17:16
@mcmire
Copy link
Contributor Author

mcmire commented Jul 29, 2022

simplify-monorepo-workflow-tests needs to be merged into this and conflicts need to be resolved. Done!

@mcmire mcmire force-pushed the simplify-monorepo-workflow-tests branch 4 times, most recently from 3db4897 to c30ce5c Compare August 22, 2022 21:19
Base automatically changed from simplify-monorepo-workflow-tests to main August 25, 2022 21:42
This commit changes how the root version is bumped per release so that
instead of looking like `<yyyy>.<mm>.<dd>`, it looks like
`<yyyy><mm><dd>.<release-number>.0`. That is, the root version
consists of the date that the release was created + a release number,
which is incremented every time a new release is issued.
@mcmire mcmire force-pushed the include-build-number-in-version branch from c703a77 to d422085 Compare August 25, 2022 22:30
});
});
});

it('collects information about a polyrepo project', async () => {
Copy link
Contributor Author

@mcmire mcmire Aug 25, 2022

Choose a reason for hiding this comment

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

I decided to drop the polyrepo test from this because readProject hasn't technically supported polyrepo projects and that is more true with this PR than before. I think this is something we can fix in a later PR.

@mcmire mcmire marked this pull request as ready for review August 25, 2022 22:44
@mcmire
Copy link
Contributor Author

mcmire commented Aug 25, 2022

This is ready for review.

@mcmire
Copy link
Contributor Author

mcmire commented Sep 9, 2022

I wasn't able to get to this today. I may have some time this weekend to make these changes but can't promise anything. Sorry :(

import { readCommandLineArguments } from './command-line-arguments';
import { readProject, Project } from './project';

interface InitialParameters {
project: Project;
tempDirectoryPath: string;
reset: boolean;
today: Date;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There's no need to read the date from TODAY nor pass this down to anywhere, as a date isn't used anymore to calculate the version string. So it gets removed from everywhere.

@mcmire
Copy link
Contributor Author

mcmire commented Sep 19, 2022

This is now updated as per discussion around the release version.

Co-authored-by: Mark Stacey <[email protected]>
Copy link
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 69c81e8 into main Sep 22, 2022
@mcmire mcmire deleted the include-build-number-in-version branch September 22, 2022 19:34
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.

Monorepo independent flow: Include build number in release version
2 participants