Skip to content

Bump develop when release branch is created #744

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

Conversation

JakeGinnivan
Copy link
Contributor

// @GeertvanHorrik @o-o00o-o, @judgeaxl, @asbjornu

Keen on feedback. I think we can change develops tag to alpha, but I think maybe this should go into v4 and I try get an alpha of that out asap as this is really a breaking change.

 - Fixed an commit counting bug
 - Based on discussion at GitTools#632
 - Fixes GitTools#695
 - Updated related docs
@JakeGinnivan
Copy link
Contributor Author

This is going into v4..

@JakeGinnivan JakeGinnivan deleted the BumpDevelopWhenReleaseBranchCreated branch January 26, 2016 02:39
@GeertvanHorrik
Copy link
Contributor

This is a fantastic feature, great to have!

Not sure if this PR also includes using alpha instead of unstable for the develop branch? If so, I think the images should be updated as well.

The rest looks really good, great work.

@asbjornu
Copy link
Member

@JakeGinnivan This looks great, but I'm wondering why you've closed it and deleted the branch?

@JakeGinnivan
Copy link
Contributor Author

Closed it out, am forward merging master into the v4 release branch then I will rebase this on top of that.

@JakeGinnivan
Copy link
Contributor Author

Will reopen over the next few days. I forgot that I pulled a bunch of stuff into GitTools.Core in the v4 branch, which now I have to move the changes on master into Core

@tibitoth
Copy link

tibitoth commented Feb 18, 2019

a forgotten bug

@asbjornu
Copy link
Member

@totht91: If you need this and what @JakeGinnivan writes about the changes having moved to GitTools.Core is incorrect, please rebase the changes on the current HEAD of master and submit a pull request.

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.

4 participants