Skip to content
This repository has been archived by the owner on Mar 9, 2022. It is now read-only.

Always require a RELEASE_TYPE when creating a release #248

Merged
merged 3 commits into from
Feb 16, 2018
Merged

Conversation

wennmo
Copy link
Member

@wennmo wennmo commented Feb 16, 2018

The release.sh used for creating Mira releases assumed a patch release if a type was not given.

This could lead to unwanted releases if e.g. running ./release.sh ?. Removed the assumption of patch and the script will now exit if RELEASE_TYPE is not set or is invalid, and also print usage instructions.

@FredrikFolkesson
Copy link
Contributor

This could lead to unwanted releases if e.g. running ./release.sh ?

Who would make such a silly misstake? :P

Copy link
Contributor

@FredrikFolkesson FredrikFolkesson left a comment

Choose a reason for hiding this comment

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

LGTM!

@wennmo wennmo merged commit f716dab into master Feb 16, 2018
@wennmo wennmo deleted the release_script branch February 16, 2018 09:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants