Skip to content

ci(actions): remove deprecated runtime warnings [Codex] - #1361

Open
jderochervlk wants to merge 1 commit into
masterfrom
jvlk/github-actions-updates
Open

jderochervlk wants to merge 1 commit into
masterfrom
jvlk/github-actions-updates

Conversation

@jderochervlk

Copy link
Copy Markdown
Collaborator

GitHub Actions was annotating deploy runs because several third-party actions still targeted Node.js 20. Jobs using ubuntu-latest also emitted notices ahead of the Ubuntu 26 migration.

This upgrades Wrangler, sticky PR comments, and artifact upload to Node.js 24-compatible releases. All jobs are pinned to Ubuntu 24.04 so the runner environment remains stable and migration notices stop.

Deployment branch names now enter shell steps through environment variables, preserving URL behavior while avoiding direct interpolation of untrusted GitHub context.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 18, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-18T14:29:03.707124Z db16fd3 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

Copy link
Copy Markdown
Contributor

Guide Cloudflare deployment

⛅️ wrangler 4.130.0
────────────────────
🌀 Building list of assets...
✨ Read 17 files from the assets directory /home/runner/work/rescript-lang.org/rescript-lang.org/apps/guide/build/client
🌀 Starting asset upload...
No updated asset files to upload. Proceeding with deployment...
Total Upload: 0.31 KiB / gzip: 0.22 KiB
Worker Startup Time: 0 ms
Uploaded guide (1.45 sec)
Worker Version ID: 8e7fef28-d72e-404c-b04b-7a253733bd79
Version Preview URL: https://8e7fef28-guide.rescript.workers.dev
Version Preview Alias URL: https://jvlk-github-actions-updates-guide.rescript.workers.dev

To deploy this version to production traffic use the command wrangler versions deploy

Changes to non-versioned settings (config properties 'logpush' or 'tail_consumers') take effect after your next deployment using the command wrangler versions deploy

Changes to triggers (routes, custom domains, cron schedules, etc) must be applied with the command wrangler triggers deploy

@github-actions

Copy link
Copy Markdown
Contributor

Cloudflare deployment

Deployement ID: 8cfbc255-702e-43ad-9be1-0f52304aad06
Deployment Environment: preview

⛅️ wrangler 4.130.0
────────────────────
✨ Compiled Worker successfully
Uploading... (8394/8880)
Uploading... (8556/8880)
Uploading... (8718/8880)
Uploading... (8880/8880)
✨ Success! Uploaded 486 files (8394 already uploaded) (5.63 sec)

✨ Uploading _redirects
✨ Uploading Functions bundle
🌎 Deploying...
✨ Deployment complete! Take a peek over at https://8cfbc255.rescript-lang.pages.dev
✨ Deployment alias URL: https://jvlk-github-actions-updates.rescript-lang.pages.dev

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 18, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
guide db16fd3 Commit Preview URL

Branch Preview URL
Sep 18 2026, 02:28 PM

@jderochervlk
jderochervlk enabled auto-merge (squash) September 18, 2026 14:09
Upgrade JavaScript actions to Node.js 24-compatible releases and pin hosted runners to Ubuntu 24.04. Pass branch names through the step environment so deployment scripts do not interpolate untrusted context directly.
@github-actions

Copy link
Copy Markdown
Contributor

Docs preview

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