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

adjust workflows to new branch structure #199

Closed
wants to merge 2 commits into from

Conversation

taukakao
Copy link
Member

@taukakao taukakao commented Aug 5, 2024

The changes make it possible to build main with the Vib Workflow by automatically changing the :dev tag to :main during building.

They also add a workflow to easily sync the main branch with the dev branch on demand, while checking if core image is up to date as well.

When this is trough, I will add these changes to the other images as well.

@taukakao taukakao marked this pull request as draft August 5, 2024 15:25
@taukakao taukakao marked this pull request as ready for review August 5, 2024 15:28
Copy link
Member

@kbdharun kbdharun left a comment

Choose a reason for hiding this comment

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

Hi, This must be addressed before merging:

The main branch has branch protection so direct commits won't work from GitHub actions so this must be updated to create a PR instead (since removing it isn't good practice). We can discuss other possible alternatives too.

@taukakao taukakao marked this pull request as draft August 5, 2024 16:07
@taukakao
Copy link
Member Author

taukakao commented Aug 5, 2024

Seems that implementing this correctly is pretty much impossible due to branch protection rules.

@taukakao taukakao mentioned this pull request Aug 5, 2024
@taukakao taukakao closed this Aug 5, 2024
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.

3 participants