Skip to content

chore: add dprint to pnpm allowBuilds#968

Merged
graphite-app[bot] merged 1 commit into
mainfrom
c/05-18-chore_add_dprint_to_pnpm_allowbuilds_
May 18, 2026
Merged

chore: add dprint to pnpm allowBuilds#968
graphite-app[bot] merged 1 commit into
mainfrom
c/05-18-chore_add_dprint_to_pnpm_allowbuilds_

Conversation

@camc314

@camc314 camc314 commented May 18, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@camc314 camc314 marked this pull request as ready for review May 18, 2026 07:36
Copilot AI review requested due to automatic review settings May 18, 2026 07:36

camc314 commented May 18, 2026

Copy link
Copy Markdown
Contributor Author

How to use the Graphite Merge Queue

Add the label 0-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds pnpm build-script approval configuration so dprint can run its allowed build step during dependency installation.

Changes:

  • Adds pnpm-workspace.yaml.
  • Allows builds for the dprint package.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@camc314 camc314 self-assigned this May 18, 2026
@camc314 camc314 added 0-merge and removed 0-merge labels May 18, 2026
@graphite-app

graphite-app Bot commented May 18, 2026

Copy link
Copy Markdown

Merge activity

@graphite-app graphite-app Bot force-pushed the c/05-18-chore_add_dprint_to_pnpm_allowbuilds_ branch from cffad54 to 0e575fa Compare May 18, 2026 08:00
@graphite-app graphite-app Bot merged commit 0e575fa into main May 18, 2026
9 checks passed
@graphite-app graphite-app Bot deleted the c/05-18-chore_add_dprint_to_pnpm_allowbuilds_ branch May 18, 2026 08:08
camc314 added a commit that referenced this pull request Jun 3, 2026
#968 added PNPM workspace configuration and updated CI commands to use
`--ignore-workspace`. The justfile still runs pnpm commands as before,
causing pnpm to ignore the nested `e2e` package since it's not listed in
the workspace. Because of that `just test` fails due to missing `vitest`
on a fresh clone.

Co-authored-by: Cameron <cameron.clark@hey.com>
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.

2 participants