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

Don't try to publish PR previews #61

Merged
merged 2 commits into from
Aug 6, 2024
Merged

Don't try to publish PR previews #61

merged 2 commits into from
Aug 6, 2024

Conversation

vcarl
Copy link
Member

@vcarl vcarl commented Aug 6, 2024

We didn't actually use them and it added a false negative for anyone without org permission to publish to the container registry under Reactiflux

Copy link

what-the-diff bot commented Aug 6, 2024

PR Summary

  • Enhanced Workflow Triggers
    The trigger conditions for the job "Build and push Docker images" in our automated workflows have been expanded. It will not just respond to changes on 'main' and 'develop' branches, but will also get activated when any modifications are done in the 'feature/actions' branch. This allows more frequent builds and testing, leading to better quality control.

  • Removed 'todo' File
    The 'todo' file, which is no longer necessary for the project, has been removed. This decluttering makes our codebase cleaner and easier to understand.

@vcarl vcarl merged commit 8bdab11 into main Aug 6, 2024
5 checks passed
@vcarl vcarl deleted the vc-build branch August 6, 2024 18:02
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.

1 participant