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

Support branch "create" action #3052

Closed
wants to merge 1 commit into from

Conversation

quantonganh
Copy link

@quantonganh quantonganh commented Jan 7, 2021

go.mod Outdated
@@ -75,3 +75,5 @@ require (
replace github.com/h2non/gock => gopkg.in/h2non/gock.v1 v1.0.14

go 1.13

replace github.com/drone/go-scm => github.com/quantonganh/go-scm v1.8.1-0.20210107022054-cb9eef77a93f

Choose a reason for hiding this comment

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

This should be removed prior to merging (but after blocking PR is merged)

Copy link
Author

Choose a reason for hiding this comment

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

Of course. It's temporary to make the test passed.

@quantonganh
Copy link
Author

As @bradrydzewski pointed out, no need to add a new field to know if a branch has been created. We can do that by checking if the before commit is empty. Updated my PR.

@tphoney
Copy link

tphoney commented Jul 1, 2021

Hi @quantonganh i am unsure of this newly added behaviour, as it changes what existing drone users would expect. To move this forward in an open way i would suggest adding a proposal here https://github.com/drone/proposal. So we can design with the rest of the community what the behaviour should be and if the community as a whole needs this functionality.

For now i am going to close this PR. Thank you for your submission and the time and effort you have put in.

@tphoney tphoney closed this Jul 1, 2021
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.

3 participants