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

Set fixed in when a PR is merged #131

Merged
merged 10 commits into from Jul 17, 2019
Merged

Set fixed in when a PR is merged #131

merged 10 commits into from Jul 17, 2019

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Jul 12, 2019

Replacement of #106. Still untested and needs code to authenticate against Redmine.

@ekohl
Copy link
Member Author

ekohl commented Jul 12, 2019

@tbrisker can you have a look at the general workflow in the last commit? I do intend to parse x.y-stable and find the next open x.y.z version as well for cherry picks but first the basics.

@tbrisker
Copy link
Member

Looks reasonable to me. I'd be fine with setting the version for cherry-picks manually, since they require special consideration anyways. different projects handle them differently - some do a big cherry-pick PR prior to a new z-stream, some do multiple smaller PRs during the -stable lifetime and some push directly to the stable branch. This would already save the manual work of having to set fixed-in for the large majority of PRs.
I wonder if we could infer the version from some version file in the commit? although that is different by project as well.

@ekohl ekohl force-pushed the set-fixed-in branch 2 times, most recently from 7d58e97 to c4a65af Compare July 15, 2019 17:41
@ekohl ekohl marked this pull request as ready for review July 15, 2019 18:26
Some Redmine projects contain multiple projects and prefix their
versions as foreman_plugin-1.2.3 and smart_proxy_plugin-2.3.4. This adds
the redmine_version_prefix in the config which allows doing so. It also
only considers versions that start with a number because that allows
sorting. It will also skip versions such as 'Backlog'.
Copy link

@zjhuntin zjhuntin left a comment

Choose a reason for hiding this comment

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

This looks great! Is there a way that I can help test the PR?

@ekohl
Copy link
Member Author

ekohl commented Jul 16, 2019

It's hard because a lot depends on setup. To run an instance, need to create a GH app, deploy it an instance and set all the secrets up. I have one at https://github.com/ekohl/gh-test-env and verified it with one issue. I'll need to check again on a project where we combine multiple git repositories though and verify it ends up in the correct versions.

@ekohl ekohl merged commit 267b699 into theforeman:app Jul 17, 2019
@ekohl ekohl deleted the set-fixed-in branch June 22, 2021 13:40
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.

None yet

3 participants