Skip to content
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

fix(build): main migration #121

Merged
merged 1 commit into from
Feb 10, 2021
Merged

fix(build): main migration #121

merged 1 commit into from
Feb 10, 2021

Conversation

christiancompton
Copy link

This changeset updates the automation and documentation to support the new main default branch.

@padamstx padamstx self-requested a review February 10, 2021 21:33
Copy link
Member

@padamstx padamstx left a comment

Choose a reason for hiding this comment

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

Changes look good, but I have one more. While reviewing the PR, I was curious why no changes were needed to the .travis.yml file and discovered that we have this as part of the "deploy" stage:

    repo: IBM/node-sdk-core

Please change this line to be:

    branch: main

Discussed this with @dpopp07 and we both agree on this change. Not sure why we were using the "repo" setting to begin with :)

Edit: oh, one more thing... can you please change your commit message to something like:

fix(build): main migration

the "fix" type will cause a new patch level, which means we'll exercise the semantic release step with the branch name change, etc.

Copy link
Member

@dpopp07 dpopp07 left a comment

Choose a reason for hiding this comment

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

Looks good, but I second Phil's suggestion to go ahead use only run deploy builds on the main branch

@christiancompton christiancompton changed the title chore: main migration fix(build): main migration Feb 10, 2021
@christiancompton christiancompton merged commit 1919510 into main Feb 10, 2021
@christiancompton christiancompton deleted the main-support branch February 10, 2021 22:03
ibm-devx-automation pushed a commit that referenced this pull request Feb 10, 2021
## [2.7.2](v2.7.1...v2.7.2) (2021-02-10)

### Bug Fixes

* **build:** main migration ([#121](#121)) ([1919510](1919510))
@ibm-devx-automation
Copy link

🎉 This PR is included in version 2.7.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants