Skip to content

Allow skipping packages in a release #1505

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 4 commits into from
Jan 31, 2019
Merged

Allow skipping packages in a release #1505

merged 4 commits into from
Jan 31, 2019

Conversation

Feiyang1
Copy link
Member

Reason:
Lerna doesn't work well with merge commits and may mark a package as changed even though it's not changed except for the version bump from the last release.

Background:
We introduced a release branch as a staging area for the next release while development continues in the master branch. Eventually we publish the packages from the release branch and make the version bumps in package.json and tags. It creates a merge commit in the master branch when we merge the release branch back.

Copy link
Contributor

@mikelehen mikelehen left a comment

Choose a reason for hiding this comment

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

LGTM though I'm sorry you ran into this with the release branch stuff. :-( Thanks for the thorough comment explaining the context. Hopefully lerna provides a better solution in the future.

@mikelehen mikelehen removed their assignment Jan 30, 2019
Copy link
Contributor

@hiranya911 hiranya911 left a comment

Choose a reason for hiding this comment

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

LGTM.

@Feiyang1 Feiyang1 merged commit 0c313d2 into master Jan 31, 2019
@Feiyang1 Feiyang1 deleted the fei-releaseupd branch January 31, 2019 02:52
@firebase firebase locked and limited conversation to collaborators Oct 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants