Skip to content

Conversation

@sandhose
Copy link
Member

@sandhose sandhose commented Jan 6, 2025

This unifies a few installs we're doing, and will make sure we get up-to-date tools. I'm tempted to pin the action, but that would probably induce a lot of dependabot update for it, as any tool update means they release a patch version (even for tools we don't use)

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 6, 2025

Deploying matrix-authentication-service-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: ede55c2
Status: ✅  Deploy successful!
Preview URL: https://1f811641.matrix-authentication-service-docs.pages.dev
Branch Preview URL: https://quenting-ci-install-action.matrix-authentication-service-docs.pages.dev

View logs

@sandhose sandhose force-pushed the quenting/ci-install-action branch from 2fc6ea9 to 3cb02d1 Compare January 6, 2025 09:57
@sandhose sandhose force-pushed the quenting/ci-install-action branch from 3cb02d1 to ede55c2 Compare January 6, 2025 10:00
@sandhose sandhose added A-CI Related to continuous integration/GitHub Actions T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. labels Jan 6, 2025
@sandhose sandhose requested a review from reivilibre January 16, 2025 14:58
Copy link
Contributor

@reivilibre reivilibre left a comment

Choose a reason for hiding this comment

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

works for me.

I thought I recognised the name somewhere: the author is also the maintainer of pin-project!

@sandhose sandhose merged commit 41fca68 into main Jan 16, 2025
23 checks passed
@sandhose sandhose deleted the quenting/ci-install-action branch January 16, 2025 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CI Related to continuous integration/GitHub Actions T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants