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

Update Release Drafter template for project workflows #4025

Merged
merged 14 commits into from
Jun 15, 2021

Conversation

robwalch
Copy link
Collaborator

@robwalch robwalch commented Jun 8, 2021

  • Update Release Drafter template for project workflows
  • Move step into build.yml

@robwalch robwalch requested a review from tjenkinson June 9, 2021 00:16
@tjenkinson
Copy link
Member

Pushed a few tweaks so that it now sets the tag in the notes to the one that is being published to npm, and also uploads the release zip to the draft release.

It's difficult to test so we'll just have to see if it works when it's merged 😆

@tjenkinson
Copy link
Member

I think now it will use the PR labels to set the version in the draft release, but use the one from the actual tag when the tag is pushed

$CHANGES

## Demo Page
> Get demo url from https://github.com/video-dev/hls.js/tree/deployments
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Is there an env var for the netlify demo URL?

Copy link
Member

Choose a reason for hiding this comment

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

I don't think it's possible to get custom variables in yet looking at release-drafter/release-drafter#847

although that would be a neat feature, so I might have a look at that at some point

@robwalch
Copy link
Collaborator Author

robwalch commented Jun 9, 2021

Thanks for the tweaks @tjenkinson!

although it is still possible to remove the skip-changelog from rennovate PRs if there's something worthy of flagging in release notes
@tjenkinson
Copy link
Member

@robwalch shall we merge this? It's not possible to get the commit demo url in the message at the moment without action changes which I'm looking into separately

Comment on lines -229 to -237
- name: create github release
id: create_release
uses: actions/create-release@v1
with:
tag_name: ${{ github.ref }}
release_name: ${{ github.ref }}
draft: true
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Copy link
Collaborator Author

@robwalch robwalch Jun 15, 2021

Choose a reason for hiding this comment

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

@tjenkinson Is actions/create-release no longer needed?

Will release-drafter/release-drafter include all the same assets (release zip)? Ah I see your tweaks cover the zip file.

@robwalch robwalch merged commit bf428a7 into master Jun 15, 2021
@robwalch robwalch deleted the task/release-drafter-2 branch June 15, 2021 16:18
@tjenkinson
Copy link
Member

Looks like it worked. I just manually added the skip-change-log label to all the rennovate updates since the last release too so they should disappear on the next build

@robwalch robwalch added this to the 1.0.9 milestone Nov 10, 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.

None yet

2 participants