Skip to content

Add condition to skip release and test release workflows from forks#31

Merged
Stormheg merged 2 commits intodjango-commons:mainfrom
browniebroke:skip-release-from-forks
Mar 19, 2026
Merged

Add condition to skip release and test release workflows from forks#31
Stormheg merged 2 commits intodjango-commons:mainfrom
browniebroke:skip-release-from-forks

Conversation

@browniebroke
Copy link
Copy Markdown
Member

After doing a release, when I synced my fork from the command line and pushed the new tag, the release workflow ran (and failed). There is no point running the release or test release workflows from forks.

This PR is adding a condition to skip the initial job of each workfow. Following jobs will be skipped automatically due to dependencies.

Copy link
Copy Markdown
Member

@Stormheg Stormheg left a comment

Choose a reason for hiding this comment

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

Thanks!

Would it make sense to briefly explain in a comment why the check is there?

@browniebroke
Copy link
Copy Markdown
Member Author

Would it make sense to briefly explain in a comment why the check is there?

Good idea

Co-authored-by: Bruno Alla <browniebroke@users.noreply.github.com>
@Stormheg Stormheg merged commit 005d20d into django-commons:main Mar 19, 2026
2 checks passed
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