Skip to content

devex: prepr:web and app with proper caching #4957

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.

Sign up for GitHub

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

Jump to bottom
Open
IMB11 wants to merge 5 commits into
base: main
Choose a base branch
from
Open

devex: prepr:web and app with proper caching #4957

IMB11 wants to merge 5 commits into from
+19 −17

Conversation

@IMB11
Copy link
Member

@IMB11 IMB11 commented Dec 23, 2025

This PR changes the pre-pr/pre-commit flow for frontend changes:

  • Rather than doing pnpm web:fix etc. for all the projects touched, you can now just do pnpm prepr:frontend:lib, pnpm prepr:frontend:web or pnpm prepr:frontend:app
  • Caching has been re-enabled for the fix and intl:extract endpoints, cache is only invalidated if something changes in a package, or if the tooling-config package has been modified.

@IMB11 IMB11 requested a review from a team December 23, 2025 23:32
@IMB11 IMB11 enabled auto-merge December 23, 2025 23:39
@IMB11 IMB11 added frontend Involves work from the frontend team dev-ex Improvements to developer experience labels Dec 23, 2025
@Prospector
Copy link
Member

Prospector commented Dec 24, 2025

github workflow needs updating since intl:extract is gone

@tdgao
Copy link
Contributor

tdgao commented Dec 24, 2025

Tested and works well for me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tdgao tdgao tdgao approved these changes

Assignees

No one assigned

Labels

dev-ex Improvements to developer experience frontend Involves work from the frontend team

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@IMB11 @Prospector @tdgao