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

Trigger maven CI action upon push to main #1053

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

fkleedorfer
Copy link
Collaborator

The last change to the maven CI action removed the push trigger. This resulted in the action not being executed on the main branch when a PR was rebased onto main.

This PR reintroduces the push event as a trigger and limits it to the main branch, so it does not trigger the action upon push to any branch. Hopefully this leads to the action running on the main branch after a PR merge. Direct pushes to main are not allowed, so there should not be any other effect.

Copy link
Collaborator

@steveraysteveray steveraysteveray left a comment

Choose a reason for hiding this comment

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

What did you think of my suggestion of providing a manual invocation of the validation check for any branch

@steveraysteveray steveraysteveray merged commit 67f819f into main Dec 5, 2024
1 check passed
@steveraysteveray steveraysteveray deleted the trigger-maven-ci-upon-push-to-main branch December 5, 2024 11:08
@fkleedorfer
Copy link
Collaborator Author

Ah sorry, forgot about that - we can do this... PR coming up

@fkleedorfer
Copy link
Collaborator Author

PR is here #1054

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