Skip to content

Push to main warning#45

Merged
briangow merged 1 commit into
mainfrom
bg_add_main_warning
Jun 30, 2025
Merged

Push to main warning#45
briangow merged 1 commit into
mainfrom
bg_add_main_warning

Conversation

@briangow

Copy link
Copy Markdown
Collaborator

Currently the active development on this repository is being done against the dev branch. Once those changes are finalized, we will merge dev into main.

This PR adds a workflow to warn users against opening PRs against main. This is primarily to prevent those of us who are doing the active development against dev from mistakenly merging our changes into main.

If you notice an issue in main that needs to be fixed soon, you can still open a PR against main and we will merge the update into dev as well, if needed.

@briangow briangow requested a review from tompollard June 30, 2025 13:23
@github-actions

Copy link
Copy Markdown

⚠️ Warning: This PR targets the main branch.
Please ensure this is intentional. Current development on main has been paused.
A major update is being worked on against the dev branch and will be merged into
main once ready.

@tompollard tompollard left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

looks good to me!

@tompollard

Copy link
Copy Markdown
Collaborator

I'd also suggest changing the default branch to dev (let me know if you'd like to do this and you don't have permissions yourself).

@briangow briangow merged commit 460a739 into main Jun 30, 2025
1 check passed
@briangow

Copy link
Copy Markdown
Collaborator Author

@tompollard , I don't think I have the needed permission to change the default branch to dev. I'm fine doing that, so if you have permission please feel free to do so.

@tompollard tompollard deleted the bg_add_main_warning branch June 30, 2025 14:06
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