Skip to content

Move all dependencies on the workspace level #4656

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

Merged
merged 3 commits into from
Jun 11, 2025

Conversation

sandhose
Copy link
Member

This moves all the dependencies to the workspace level, to make it less likely to conflict on upgrades, as well as to make sure we enable the same set of features for all our dependencies, avoiding unnecessary recompilations.

Copy link

cloudflare-workers-and-pages bot commented Jun 10, 2025

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

Latest commit: 1546318
Status: ✅  Deploy successful!
Preview URL: https://c3080518.matrix-authentication-service-docs.pages.dev
Branch Preview URL: https://quenting-dependencies-in-wor.matrix-authentication-service-docs.pages.dev

View logs

@sandhose sandhose force-pushed the quenting/dependencies-in-workspace branch 2 times, most recently from b3a0112 to 830eeb0 Compare June 10, 2025 11:56
@sandhose sandhose force-pushed the quenting/dependencies-in-workspace branch from 830eeb0 to 1546318 Compare June 10, 2025 12:25
@sandhose sandhose requested a review from reivilibre June 11, 2025 08:47
@sandhose sandhose added the T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. label Jun 11, 2025
@sandhose sandhose merged commit 70366d1 into main Jun 11, 2025
28 checks passed
@sandhose sandhose deleted the quenting/dependencies-in-workspace branch June 11, 2025 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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