-
Notifications
You must be signed in to change notification settings - Fork 95
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
Fix deploy script to not attempt re-publishing the same version of a plugin #124
Fix deploy script to not attempt re-publishing the same version of a plugin #124
Conversation
Co-authored-by: Joe McGill <[email protected]>
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.
Thanks for accepting my suggestion, @mukeshpanchal27. This looks good to me!
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.
Thanks @mukeshpanchal27, looks great!
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.
@mukeshpanchal27 Thanks for your work. LGTM!
Description of the Change
I tried this solution in folk repo and here is the result for it. At the moment the WebP Uploads is release with version
1.0.0
webp-uploads
and version1.0.0
. It will show the message "ℹ︎ Version 1.0.0 of plugin webp-uploads was already published" and skip the script execution without throwing any errors. Check result herewebp-uploads
and version1.0.1
. It will execute all the steps and commit the new version to the SVN repository(For testing i have comment the SVN commit step). Check result hereCloses #123
Credits
Props @mukeshpanchal27 @joemcgill @felixarntz
Checklist: