jenkins-infra / github-reusable-workflows

Repository for reusable workflows
MIT License
1 stars 6 forks source link

Use `steps.draft.outputs.body` #3

Closed jglick closed 2 years ago

jglick commented 2 years ago

https://github.com/jenkins-infra/github-reusable-workflows/pull/1#issuecomment-1135852603 alternative to #2 using https://github.com/jenkins-infra/interesting-category-action/pull/5 as per https://github.com/jenkins-infra/github-reusable-workflows/pull/2#issuecomment-1145526728.

jglick commented 2 years ago

@jetersen squash-merged #1, making this more complicated…

jetersen commented 2 years ago

git pull --rebase origin main makes it super easy 🤔

jglick commented 2 years ago

I used some git-rebase variant but it was complicated by the base branch going away.

jglick commented 2 years ago

https://github.com/jenkinsci/jenkins-infra-test-plugin/pull/23 was correctly ignored; now checking that https://github.com/jenkinsci/jenkins-infra-test-plugin/pull/24 correctly triggers a release.

jetersen commented 2 years ago

https://github.com/jenkinsci/jenkins-infra-test-plugin/runs/6869048725?check_suite_focus=true seems to work @jglick

jetersen commented 2 years ago

Fixed the build failure by using a version bom that had 2.249.x bom:

https://github.com/jenkinsci/jenkins-infra-test-plugin/commit/49b5815e8158d5e5db9ee94039f23d12c1a7e63c my bad for not making a PR 😐 Though this commit still triggered a release so win/win? 😅 https://github.com/jenkinsci/jenkins-infra-test-plugin/releases/tag/69.v49b_5815e8158

jglick commented 2 years ago

https://github.com/jenkinsci/bom/releases?page=3 huh, I seem to have picked a version from my local repo that was in incrementals but not releases. At least this demonstrated that https://github.com/jenkins-infra/jenkins-maven-cd-action/issues/3 is fixed!

jglick commented 2 years ago

this commit still triggered a release

Yes, any commit could trigger a release if the currently unreleased changes include at least one PR with an interesting label.

jetersen commented 2 years ago

@jglick you should be able to use the github's release page on https://github.com/jenkins-infra/interesting-category-action/releases to draft a new release and pick a tag like v1.2.0 ? 👏 EDIT: whelp just as fast I as I could type it, you created a release 😆

jglick commented 2 years ago

use the github's release page

Actually easier:

gh release create v1.2.0 --generate-notes
jetersen commented 2 years ago

@jglick cool! Did not know that CLI options existed!

jglick commented 2 years ago

So what is the next step here? We need an initial release tag. Then I can update the test plugin to use that as a final sanity check. Then we should amend the workflow template in .github to suggest it, and start rolling it out to various plugins?

jetersen commented 2 years ago

Yea @jglick :)

I suggest we create a v1.0.0 and a v1

and than move v1 along v1.x.x and revert v1 if we have any bad v1.x.x releases.