-
Notifications
You must be signed in to change notification settings - Fork 640
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
GitFlow: Include commits in PreReleaseTag for release, hotfix and feature branches #323
Comments
The problem is there are two styles of consuming builds, continuous deployment and continuous delivery. Continuous deployment means you want each build to have a unique nuget version, continuous delivery means you do not. When I am creating a release, I want to commit multiple times/submit multiple pull requests and when i am ready hit the deploy button which deploys the current version and then tags the vcs with that version. Maybe we need to release v2, then level up the config in v2.1 and support something like:
This could actually simplify GitVersion by allowing us to simply support default conventions for different branch types via config and we can delete a heap of the hard-coded branch logic. |
@JakeGinnivan I see the problem Good idea! I think I will submit a work in progress PR to discuss about the implementation |
There are a couple times when we have done this that we want to deploy (manually) a Pull Request build just to test out a few things before pulling into the develop branch. This works great the first time that you do this, but if you find that something is not quite right and go back and fix it, you can't then deploy the corrected pull request version, as the version already exists. In order to do it, we have to manually go and hack Octopus Deploy to delete the release, and associated nupkg packages, so that we can deploy again. So it would definitely be nice if "something" can be done here to help with this edge case. |
I thought that there was a version number that contained the commit count in the fourth build number that could be used specifically for generating nuget packages that increased in number every commit?
|
@robdmoore that is true. We do have variables, but this would enable different branches to version differently. I.e master can still be cont delivery and others are continuous deployment I also think it is an easier concept than variables |
The new config ideas could allow you to say pull request branches are not semver and use 4 part versions. What do you think of the suggested approach Sent from my Windows Phone From: Gary Ewan Parkmailto:notifications@github.com When I am creating a release, I want to commit multiple times/submit multiple pull requests and when i am ready hit the deploy button which deploys the current version and then tags the vcs with that version. There are a couple times when we have done this that we want to deploy (manually) a Pull Request build just to test out a few things before pulling into the develop branch. This works great the first time that you do this, but if you find that something is not quite right and go back and fix it, you can't then deploy the corrected pull request version, as the version already exists. In order to do it, we have to manually go and hack Octopus Deploy to delete the release, and associated nupkg packages, so that we can deploy again. So it would definitely be nice if "something" can be done here to help with this edge case. — |
Ah, interesting idea, I hadn't concerned it from that side of things. Yes, I can see how that would work! I definitely like the approach of everything being described/configured in the yaml file, that way it becomes an artefact of the repository it is in. |
I prefer to have the option to override the InformationalVersion in the GetVersionConfig.yaml and command-line, just like #315. I think this could solve this issue and also some other issues. If I know have to insert the version myself into .nuspec to override this behavior. |
This config approach is done. Check out the 3.0.0 branch @remyvd I think that is a separate issue. Mind opening a new issue if it is still a problem with 3.0.0 |
For release/hotfix branches currently the PreReleaseTag looks like this:
and for feature-branches
The problem is that for every commit, the NuGet version will be the same. My proposition is to do the same as on the develop branch: Add the number of commits (build metadata) in the pre-release tag.
The change in the code will be trivial, but I want to hear your opinion, before I send a PR.
The text was updated successfully, but these errors were encountered: