-
Notifications
You must be signed in to change notification settings - Fork 807
Update GitHub Actions to run all CI #1231
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
Update GitHub Actions to run all CI #1231
Conversation
Signed-off-by: Joe Adams <github@joeadams.io>
45a0686 to
0cd681a
Compare
Signed-off-by: Joe Adams <github@joeadams.io>
This will stop the CI jobs but leave existing PRs in place until we can get those updated. Signed-off-by: Joe Adams <github@joeadams.io>
| @@ -1,2 +1,2 @@ | |||
| # Adding queries to this file is deprecated | |||
| # Example queries have been transformed into collectors. | |||
| # Example queries have been transformed into collectors. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just a formatting fix that CI won't pass without
| @@ -1,103 +1,33 @@ | |||
| --- | |||
| version: 2.1 | |||
| # Prometheus has switched to GitHub action. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me know if we should just delete this file entirely. I think we need it short term while we get PRs merged or rebased.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea, we can do this in 2 phases. First get this working, then stop CircleCI, then delete the file.
|
I removed the CircleCI branch protections. |
Bonus points - this runs much faster than in circle