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

ci: Use setup-sqlc@v4 action #82

Merged
merged 2 commits into from
Nov 29, 2023
Merged

Conversation

andrewmbenton
Copy link
Contributor

Just noticed that your sqlc workflow would be simpler with the setup-sqlc GitHub action.

@brandur
Copy link
Contributor

brandur commented Nov 29, 2023

@andrewmbenton Thanks!

Would you mind doing a no-op git commit --amend and force pushing again? Something odd seems to have happened with the GitHub Actions checks here — they're still in limbo an hour later and GitHub is not currently reporting any status events.

@bgentry
Copy link
Contributor

bgentry commented Nov 29, 2023

@brandur I think we might need to approve this first for a first-time contributor before the actions will run
Screenshot 2023-11-28 at 8 18 23 PM

@brandur
Copy link
Contributor

brandur commented Nov 29, 2023

@bgentry So I have this feature on for other repos too, but usually there's an "approve CI run" button you can press. I'm not sure if I'm blind or there's a bug somewhere, but I'm not seeing it anywhere in my UI. Do you see it?

@bgentry
Copy link
Contributor

bgentry commented Nov 29, 2023

I've been looking everywhere to find a setting that will make that button show up, afaict all the settings look correct but I don't see any way to approve the workflow per the docs 😖

@brandur
Copy link
Contributor

brandur commented Nov 29, 2023

WTH. I think it might be a bug ... I've seen the button hundreds of times before. It's just not here right now.

@bgentry
Copy link
Contributor

bgentry commented Nov 29, 2023

@brandur
Copy link
Contributor

brandur commented Nov 29, 2023

@bgentry Doh. Okay NM, I guess that's it. Good idea to actually RTFM.

@bgentry bgentry enabled auto-merge (squash) November 29, 2023 02:45
@bgentry bgentry merged commit cccd3f6 into riverqueue:master Nov 29, 2023
@bgentry
Copy link
Contributor

bgentry commented Nov 29, 2023

Thank you @andrewmbenton 🙏

@andrewmbenton
Copy link
Contributor Author

Thank you @andrewmbenton 🙏

Happy to help work out the configuration kinks early on.

brandur added a commit that referenced this pull request Jan 27, 2024
A small clean up in `ci.yml` to remove the `BIN_PATH` variable, which
was previously used for sqlc before #82 moved that to a GitHub Action.

It's not a particular problem that `BIN_PATH` is there, but the comment
above it that it's added to `PATH` is no longer true, so probably best
to remove it for accuracy's sake.
bgentry pushed a commit that referenced this pull request Jan 28, 2024
A small clean up in `ci.yml` to remove the `BIN_PATH` variable, which
was previously used for sqlc before #82 moved that to a GitHub Action.

It's not a particular problem that `BIN_PATH` is there, but the comment
above it that it's added to `PATH` is no longer true, so probably best
to remove it for accuracy's sake.
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