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

Replace some jenkins jobs with a GH Actions #58983

Merged
merged 6 commits into from Nov 30, 2020

Conversation

s0undt3ch
Copy link
Member

What does this PR do?

See title

@s0undt3ch s0undt3ch force-pushed the hotfix/github-actions branch 17 times, most recently from 7dbb948 to 758ab1e Compare November 19, 2020 16:40
@s0undt3ch s0undt3ch marked this pull request as ready for review November 19, 2020 16:41
@s0undt3ch s0undt3ch requested a review from a team as a code owner November 19, 2020 16:41
@s0undt3ch s0undt3ch requested review from dwoz and removed request for a team November 19, 2020 16:41
@s0undt3ch s0undt3ch changed the title Replace the pre-commit jenkins job with a GH Actions one Replace some jenkins jobs with a GH Actions Nov 19, 2020
krionbsd
krionbsd previously approved these changes Nov 19, 2020
@s0undt3ch
Copy link
Member Author

GH Actions switched to docker containers

bryceml
bryceml previously approved these changes Nov 19, 2020
@s0undt3ch s0undt3ch force-pushed the hotfix/github-actions branch 2 times, most recently from 8d0b9a7 to 50f1d09 Compare November 19, 2020 20:14
bryceml
bryceml previously approved these changes Nov 19, 2020
krionbsd
krionbsd previously approved these changes Nov 20, 2020
krionbsd
krionbsd previously approved these changes Nov 25, 2020
Copy link
Contributor

@ScriptAutomate ScriptAutomate left a comment

Choose a reason for hiding this comment

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

This is so awesome that we are migrating stuff to GitHub Actions! It's great that I can see what these actions are doing via the Check tab of the PR:

The only confirmation I'd like make is about this section where a Good, Bad, Good example set exists, though I think it is intended to have only one Good and one Bad? Just want to confirm whether it is intentional:

https://github.com/saltstack/salt/pull/58983/files?file-filters%5B%5D=.py&file-filters%5B%5D=.rst#diff-595e29dbd27a244b4837db52d7f5ffbd4dd352c4e7605b79a12921f3a122aad2R59-R80

@s0undt3ch
Copy link
Member Author

I can make it just one Good and one Bad, two good and one Bad was intentional since they refer to different things.
But we can join examples

ScriptAutomate
ScriptAutomate previously approved these changes Nov 26, 2020
@dwoz dwoz merged commit 8f6c90b into saltstack:master Nov 30, 2020
26 checks passed
@s0undt3ch s0undt3ch deleted the hotfix/github-actions branch November 30, 2020 21:44
@sagetherage sagetherage added the Aluminium Release Post Mg and Pre Si label Feb 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Aluminium Release Post Mg and Pre Si
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants