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

Modify script to allow release from non-main branch #760

Merged
merged 3 commits into from
Jul 16, 2024

Conversation

nazli-stripe
Copy link
Collaborator

Summary

Publish script changes to force devs into typing the branch name when releasing and disables releases from the main branch to promote using release branches

Motivation

KTLO

Testing

  • I tested this manually
  • I added automated tests

Documentation

Select one:

  • I have added relevant documentation for my changes.
  • This PR does not result in any developer-facing changes.

echo "Publish successful!"
echo ""

create_github_release
Copy link
Collaborator

Choose a reason for hiding this comment

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

intentionally rm'd? and is the --follow-tags not needed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

nope lost in copy paste, added back

Copy link
Collaborator

@bric-stripe bric-stripe left a comment

Choose a reason for hiding this comment

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

👍

@nazli-stripe nazli-stripe merged commit e0ef7d6 into main Jul 16, 2024
3 checks passed
nazli-stripe added a commit that referenced this pull request Jul 24, 2024
* modify script for non main release

* dont release from main

* add missing lines
@nazli-stripe nazli-stripe deleted the nazli/release-script branch September 7, 2024 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants