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

Display notice with PR link #101

Merged
merged 1 commit into from
Sep 6, 2021
Merged

Display notice with PR link #101

merged 1 commit into from
Sep 6, 2021

Conversation

Nef10
Copy link
Contributor

@Nef10 Nef10 commented Sep 6, 2021

This way the PR links are visible without opening the log of the action run.
Note: There is a limit of 10 notices per run
Demo: here

When working on this I noticed that the current version does not seem to be built correctly. When I check out develop and run a build there is a diff to the checked in dist file. This change seems to be the bump of @actions/core to 1.5.0. This version is required for this change, so please make sure to update the dependencies to the version of the lock file before building.

Fixes #100

This way the PR links are visible without opening the log of the action
run.

Note: There is a limit of 10 notices per run
@BetaHuhn BetaHuhn merged commit 0e59a9c into BetaHuhn:develop Sep 6, 2021
@BetaHuhn
Copy link
Owner

BetaHuhn commented Sep 6, 2021

Great, thanks for the PR!

@BetaHuhnBot
Copy link
Collaborator

🎉 This PR is included in version 1.13.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@Nef10 Nef10 deleted the notice-pr branch September 6, 2021 22:28
@Nef10 Nef10 mentioned this pull request Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants