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

chore(release): Set correct versions in issue template #20260

Merged
merged 2 commits into from
Aug 6, 2021

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Aug 5, 2021

What does this PR do?

When doing a release, update issue template with the versions we're tagging

Screenshot/screencast of this PR

What issues does this PR fix or reference?

#20054

How to test this PR?

Extract code to a separate file and run for example

VERSION="7.35.0"
ISSUE_TEMPLATE_FILE=".github/ISSUE_TEMPLATE/bug_report.yml"
update_issue_template "7.35.0" ".github/ISSUE_TEMPLATE/bug_report.yml"

it will update .github/ISSUE_TEMPLATE/bug_report.yml file

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Change-Id: Iebccac1611befbae19fe9b9c326b6ced918c818f
Signed-off-by: Florent Benoit [email protected]

Update latest to the version being tagged and add previous major version if not yet set

Change-Id: Iebccac1611befbae19fe9b9c326b6ced918c818f
Signed-off-by: Florent Benoit <[email protected]>
@benoitf benoitf requested a review from sparkoo as a code owner August 5, 2021 15:24
@@ -126,6 +171,9 @@ git commit -asm "${COMMIT_MSG}"
# now update ${BASEBRANCH} to the new snapshot version
git checkout "${BASEBRANCH}"

# update template in the branch
update_issue_template "${VERSION}" "${ISSUE_TEMPLATE_FILE}"
Copy link
Contributor

Choose a reason for hiding this comment

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

is the issue template file in the branch ever actually used? Can you open an issue using the branch's template?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

When cutting a .z release it won't update the template BTW (no-op)

but then for .0 release it will trigger the PR to merge to main branch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

script is creating PR to update main from a branch so we need to update the branch if we want to have the template being modified in main branch through this PR

@che-bot
Copy link
Contributor

che-bot commented Aug 5, 2021

✅ E2E Happy path tests succeed 🎉

See Details

Test product:

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.

Eclipse Che QE channel: https://mattermost.eclipse.org/eclipse/channels/eclipse-che-qe

@nickboldt nickboldt merged commit 78ad012 into eclipse-che:main Aug 6, 2021
@che-bot che-bot added this to the 7.35 milestone Aug 6, 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