Skip to content

Conversation

NathanFlurry
Copy link
Member

Changes

This was referenced Apr 24, 2025
Copy link
Member Author

NathanFlurry commented Apr 24, 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 force-pushed the 04-24-chore_fix_paths_to_sandboxed_code_execution_examples branch from b0e6205 to 25a00d4 Compare April 24, 2025 20:41
Copy link

cloudflare-workers-and-pages bot commented Apr 24, 2025

Deploying rivet-studio with  Cloudflare Pages  Cloudflare Pages

Latest commit: 34393da
Status:⚡️  Build in progress...

View logs

Copy link

cloudflare-workers-and-pages bot commented Apr 24, 2025

Deploying rivet-hub with  Cloudflare Pages  Cloudflare Pages

Latest commit: 25a00d4
Status: ✅  Deploy successful!
Preview URL: https://51440ce3.rivet-hub-7jb.pages.dev
Branch Preview URL: https://04-24-chore-fix-paths-to-san.rivet-hub-7jb.pages.dev

View logs

Copy link

cloudflare-workers-and-pages bot commented Apr 24, 2025

Deploying rivet with  Cloudflare Pages  Cloudflare Pages

Latest commit: 25a00d4
Status: ✅  Deploy successful!
Preview URL: https://39fa5b1f.rivet.pages.dev
Branch Preview URL: https://04-24-chore-fix-paths-to-san.rivet.pages.dev

View logs

@NathanFlurry NathanFlurry force-pushed the 04-24-chore_fix_paths_to_sandboxed_code_execution_examples branch from 25a00d4 to b0e6205 Compare April 24, 2025 20:58
@jog1t jog1t force-pushed the 04-24-chore_fix_paths_to_sandboxed_code_execution_examples branch from b0e6205 to 25a00d4 Compare April 24, 2025 23:44
@NathanFlurry NathanFlurry force-pushed the 04-24-chore_fix_paths_to_sandboxed_code_execution_examples branch from 25a00d4 to b0e6205 Compare April 25, 2025 09:12
@NathanFlurry NathanFlurry force-pushed the 04-24-chore_fix_paths_to_sandboxed_code_execution_examples branch from b0e6205 to 34393da Compare April 25, 2025 11:39
@NathanFlurry NathanFlurry marked this pull request as ready for review April 25, 2025 13:00
Copy link
Contributor

graphite-app bot commented Apr 25, 2025

Merge activity

  • Apr 25, 9:00 AM EDT: NathanFlurry added this pull request to the Graphite merge queue.
  • Apr 25, 9:01 AM EDT: CI is running for this pull request on a draft pull request (#2382) due to your merge queue CI optimization settings.
  • Apr 25, 9:01 AM EDT: Merged by the Graphite merge queue via draft PR: #2382.

graphite-app bot pushed a commit that referenced this pull request Apr 25, 2025
<!-- Please make sure there is an issue that this PR is correlated to. -->

## Changes

<!-- If there are frontend changes, please include screenshots. -->
@graphite-app graphite-app bot closed this Apr 25, 2025
@graphite-app graphite-app bot deleted the 04-24-chore_fix_paths_to_sandboxed_code_execution_examples branch April 25, 2025 13:01
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR reorganizes the sandboxed code execution examples by renaming directories and updating paths to better reflect their purposes, with 'eval' for code evaluation and 'standalone' for independent execution.

  • Renamed sandboxed-code-execution-js to sandboxed-code-execution-eval to clarify its code evaluation focus
  • Renamed sandboxed-code-execution-docker to sandboxed-code-execution-standalone for standalone execution examples
  • Fixed inconsistency in upload_js.ts where process.env is used instead of Deno.env
  • Removed unused uuidv4 import in upload_js.ts while crypto.randomUUID is being used
  • Fixed duplicate PROJECT constant declaration in upload_js.ts

10 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

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.

1 participant