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

Adding YAML for a GH Action (try 2) #1463

Closed
wants to merge 9 commits into from
Closed

Adding YAML for a GH Action (try 2) #1463

wants to merge 9 commits into from

Conversation

arschles
Copy link
Member

What is the problem I am trying to address?

None

How is the fix applied?

I added .github/workflows/go.yml to set up a basic GitHub action

Mention the issue number it fixes or add the details of the changes if it doesn't have a specific issue.

Ref #1460

@arschles arschles requested a review from a team as a code owner November 13, 2019 23:14
@marwan-at-work
Copy link
Contributor

@arschles actions are down lol https://www.githubstatus.com/

@arschles
Copy link
Member Author

@marwan-at-work must be why all my previous PRs weren't running 😆

I just updated the branch and looks like the action just passed. Yesterday GH opened up actions to everyone, do you think we should put this on hold until they stabilize?

@marwan-at-work
Copy link
Contributor

Probably, but also see if it provides any value that we are currently missing from our current CIs

@arschles
Copy link
Member Author

@marwan-at-work there are a few things that overall make GH actions a nicer experience for me. Most of the below points are subjective though, so I hope this doesn't sound too much like a sales pitch. My biggest overall argument for GH actions is that they're right next to the PR so you don't need to authenticate with another service or deal with another interface.

  • You can see the build logs, restart a build, etc... inside of the PR instead of going to drone/circle/etc... and logging in
  • They're a bit more modular than what I've seen in drone or circle. We can have N workflows in different yaml files, which run in parallel. We can also split our steps up into separate docker containers if they end up getting big
  • We can choose to run docker commands natively from scripts or run commands inside a container
  • We can cache our dependencies. I know we could do it with circle, but not sure about drone?
  • Scripts are already authenticated to GH and we can respond to tons of events
  • We can do testing on windows VMs if we wanted to later (docs)

Let me know what you think. I'm totes cool if we stick with drone/circle/appveyor for a while or forever if people would rather

@marwan-at-work
Copy link
Contributor

@arschles that's pretty cool, a couple things:

  • I couldn't figure out how to restart a build from the link you sent me.
  • Can we have background containers?

@arschles
Copy link
Member Author

@marwan-at-work I have pressed a restart button before, I know it, but I can't find it at the moment. Maybe it was taken out. Anyway, that might be a problem for us if it's not there.

Yes, you can run background containers with docker run -d ...

@arschles
Copy link
Member Author

Ah I see it on a different repo. There is a button called "re-run checks":

Screen Shot 2019-11-26 at 10 21 17 AM

I'm wondering if it has to do with repo permissions. I'll need to investigate this more

@arschles
Copy link
Member Author

arschles commented Dec 6, 2019

@marwan-at-work I checked some other GH actions and my guess is that you can only restart checks if they failed. See here for a failure example

@arschles arschles changed the base branch from master to main June 15, 2020 19:21
@DrPsychick
Copy link
Member

closing as we have GitHub actions already merged: https://github.com/gomods/athens/tree/main/.github/workflows

@DrPsychick DrPsychick closed this Mar 19, 2023
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.

3 participants