-
Notifications
You must be signed in to change notification settings - Fork 98
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
Auto-advance date of the next
tag
#2428
Conversation
Removes softprops/action-gh-release in favor of the `gh` CLI. Fixes web-platform-dx#2068 Closes web-platform-dx#2425
@@ -68,20 +68,33 @@ jobs: | |||
VERSION: ${{ steps.version.outputs.VERSION }} | |||
TIMESTAMP: ${{ steps.timestamp_and_hash.outputs.TIMESTAMP }} | |||
SHORT_HASH: ${{ steps.timestamp_and_hash.outputs.SHORT_HASH }} | |||
- run: npm publish --tag ${{ env.dist_tag }} | |||
- if: ${{ env.NODE_AUTH_TOKEN }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added this condition to make it easier to test on my fork.
@Elchi3 would you be interested in reviewing this? It's primarily a maintainer quality-of-life thing, to stop dismissing the Dependabot upgrade for the GitHub Action. I've tested it on my fork, so I've got fairly high confidence on this one. |
@ddbeck Hello! I am watching releases. |
Oh that sounds very annoying. Sorry about that, @szepeviktor—I'll open a PR to revert this until I find another workaround. |
Thank you. |
This produces a bunch of needless notifications for release watchers on GitHub. I think there's a way to work around this, but it needs some testing first. This reverts commit 766b40e.
This produces a bunch of needless notifications for release watchers on GitHub. I think there's a way to work around this, but it needs some testing first. This partially reverts commit 766b40e.
I've opened #2453. |
Removes softprops/action-gh-release in favor of the
gh
CLI.Fixes #2068
Closes #2425