-
Notifications
You must be signed in to change notification settings - Fork 221
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
feat(release-please): add default branch input option #202
feat(release-please): add default branch input option #202
Conversation
18a5426
to
0006e9b
Compare
@@ -46,6 +46,48 @@ describe('release-please-action', () => { | |||
}) | |||
}) | |||
|
|||
it('both opens PR to a different default branch and tags GitHub releases by default', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the test 🎉
} | ||
const releasePR = sinon.stub().returns(25) | ||
const buildStatic = sinon.stub().returns({ | ||
run: releasePR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mind adding the inverse too, that confirms we default to undefined
for the default branch? (at which point the default branch is currently detected).
@pragmaticivan thank you for the contribution, one thought about a good additional test. |
…ot provided as input
@bcoe just added a slight change on the test covering both. I felt it made more sense to complement the first test (default branch) as my change was a variation of the first one. |
@bcoe I've also added this input into the readme. Missed that part. |
Fixes https://github.com/google-github-actions/release-please-action/issues/201