-
Notifications
You must be signed in to change notification settings - Fork 71
Idea: inform when publishing the same twice #140
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
Comments
That's for sure! Thank you so much for the suggestion, would you like to send a PR? |
As much as I'd love to help out, I have more projects on my plate atm. |
For sure mate, will sort it out for you! |
Just sorted this out, will have a release soon! |
Thank you! Couldn't wish for more 😊 |
Hi @Aslemammad @webpro I have a question about the implementation of this ⬆️ if I have this
Whats suggested in this issue is that I update the template to this ⬇️ ?
I know it might be a silly question but its for an OSS repo so I wouldnt want to suggest something wrong 👀 |
With the suggested config the workflow should not be triggered for any tag. I think we have to explicitly disable the tag part. But I wouldn't worry too much about it, a mistake would be rather cheap here as this service itself is about publishing preview releases :) |
Got it @webpro thanks for your reply 🙌 |
Since the example/template has
on: [push, pull_request]
the job also runs on tags, resulting in two seemingly identical publishes for the same commit sha:Nothing wrong here, but maybe nice to info/warn something like "this version was already published".
In the same vein, the Action template could be updated to something like this:
Tags are usually published to npm anyway. Each push would still be published, though.
The text was updated successfully, but these errors were encountered: