Skip to content

Conversation

@jog1t
Copy link
Contributor

@jog1t jog1t commented Sep 2, 2025

TL;DR

Added build configuration for inspector and updated dev dependencies in Turbo config.

What changed?

  • Added a new build:inspector task with the same configuration as the existing build task
  • Added a dependency on ^build to the dev task, ensuring build tasks complete before dev starts

How to test?

  1. Run turbo run build:inspector to verify the new task works correctly
  2. Run turbo run dev to confirm it properly waits for build tasks to complete first

Why make this change?

This change improves the development workflow by ensuring that build dependencies are properly resolved before development starts. The new inspector build task allows for specialized builds with inspector capabilities while maintaining the same environment variables and output structure as the standard build.

@vercel
Copy link

vercel bot commented Sep 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
rivet-site Ready Ready Preview Comment Sep 2, 2025 1:19pm
rivet-studio Canceled Canceled Sep 2, 2025 1:19pm

Copy link
Contributor Author

jog1t commented Sep 2, 2025


How to use the Graphite Merge Queue

Add the label merge-queue 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.

@jog1t jog1t marked this pull request as ready for review September 2, 2025 13:06
@claude
Copy link

claude bot commented Sep 2, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 2, 2025

Merge activity

  • Sep 2, 1:06 PM UTC: jog1t added this pull request to the Graphite merge queue.
  • Sep 2, 1:07 PM UTC: CI is running for this pull request on a draft pull request (#2853) due to your merge queue CI optimization settings.
  • Sep 2, 1:07 PM UTC: Merged by the Graphite merge queue via draft PR: #2853.

graphite-app bot pushed a commit that referenced this pull request Sep 2, 2025
### TL;DR

Added build configuration for inspector and updated dev dependencies in Turbo config.

### What changed?

- Added a new `build:inspector` task with the same configuration as the existing build task
- Added a dependency on `^build` to the `dev` task, ensuring build tasks complete before dev starts

### How to test?

1. Run `turbo run build:inspector` to verify the new task works correctly
2. Run `turbo run dev` to confirm it properly waits for build tasks to complete first

### Why make this change?

This change improves the development workflow by ensuring that build dependencies are properly resolved before development starts. The new inspector build task allows for specialized builds with inspector capabilities while maintaining the same environment variables and output structure as the standard build.
@graphite-app graphite-app bot closed this Sep 2, 2025
@graphite-app graphite-app bot deleted the 09-02-fix_proper_turbo branch September 2, 2025 13:07
@vercel vercel bot temporarily deployed to Preview – rivet-studio September 2, 2025 13:07 Inactive
@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 2, 2025

Graphite Automations

"Test" took an action on this PR • (09/02/25)

1 assignee was added to this PR based on Kacper Wojciechowski's automation.

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