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: remove dependabot + gomod, always fetch latest #1293

Merged
merged 3 commits into from
Feb 10, 2025
Merged

ci: remove dependabot + gomod, always fetch latest #1293

merged 3 commits into from
Feb 10, 2025

Conversation

woodruffw
Copy link
Member

Instead of using Dependabot to test for an update (which was unreliable, per #1288), this always fetches the latest TSA build.

I'm not positive this is a great idea, however -- an alternative that doesn't involve blindly pulling the latest release would be to set up a release-tracking workflow that files an issue for us whenever a new TSA release is made. That would be slightly more manual, but would also avoid opaque CI changes.

Instead of using Dependabot to test, this always
fetches the latest TSA build.

I'm not positive this is a great idea, however.

Signed-off-by: William Woodruff <[email protected]>
@woodruffw woodruffw self-assigned this Jan 29, 2025
Signed-off-by: William Woodruff <[email protected]>
Copy link
Member

@jku jku left a comment

Choose a reason for hiding this comment

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

The whole workflow step is indeed one big hack but let's go with this: maybe it'll be fine until there is a real TSA.

I'll update the branch: I think you'll be able to "approve" then?

@jku
Copy link
Member

jku commented Feb 10, 2025

hmm, looks like my approval is still valid after updating. I'll merge.

@jku jku merged commit 71a7b20 into main Feb 10, 2025
25 checks passed
@jku jku deleted the ww/drop-gomod branch February 10, 2025 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants