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 CI on push to main #13

Merged
merged 2 commits into from
Dec 5, 2023
Merged

Conversation

dixonjoel
Copy link
Collaborator

@dixonjoel dixonjoel commented Dec 5, 2023

What does this Pull Request accomplish?

Trigger the CI workflow when a push to main happens.

Also, remove the 'Dump GitHub Context' step from trigger_azdo_ci since it is working correctly now.

Why should this Pull Request be merged?

The CI was not running. The same 'check_protos' is run on the PR, but in theory two separate PRs could cause conflicts. Also, the 'trigger_azdo_ci' workflow is waiting for the CI to run to be triggered.

What testing has been done?

None - will test after submit.

@dixonjoel dixonjoel merged commit bf246c3 into main Dec 5, 2023
2 checks passed
@dixonjoel dixonjoel deleted the users/jdixon/trigger-ci-on-push branch December 5, 2023 18:08
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