Skip to content

Conversation

dario-piotrowicz
Copy link
Member

Description

The changes here add a workflow that deploys the OpenNext version (deployed at https://node-test.org) of the site on every push to main, so that we can make sure that such version is always up to date with the official version.

Validation

This will be validated once merged (and merging it is very low risk)

Related Issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

@Copilot Copilot AI review requested due to automatic review settings October 7, 2025 17:16
@dario-piotrowicz dario-piotrowicz requested a review from a team as a code owner October 7, 2025 17:16
Copy link

vercel bot commented Oct 7, 2025

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

Project Deployment Preview Updated (UTC)
nodejs-org Ready Ready Preview Oct 14, 2025 8:59am

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a GitHub workflow to automatically deploy the OpenNext version of the Node.js website to Cloudflare (hosted at https://node-test.org) on every push to main. This ensures the test version stays synchronized with the official website.

  • Adds automated deployment workflow for OpenNext test site
  • Configures Cloudflare deployment using secrets for API tokens
  • Sets up proper security hardening and Node.js environment

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

codecov bot commented Oct 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.45%. Comparing base (39850b9) to head (b082c34).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8211      +/-   ##
==========================================
- Coverage   76.64%   76.45%   -0.19%     
==========================================
  Files         115      115              
  Lines        9618     9643      +25     
  Branches      322      316       -6     
==========================================
+ Hits         7372     7373       +1     
- Misses       2245     2269      +24     
  Partials        1        1              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

dario-piotrowicz and others added 2 commits October 7, 2025 19:11
Copy link
Member

@AugustinMauroy AugustinMauroy left a comment

Choose a reason for hiding this comment

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

WOW cool to the tat happened

Copy link
Member

@ovflowd ovflowd left a comment

Choose a reason for hiding this comment

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

SGTM, but other comments could be addressed

@avivkeller
Copy link
Member

@dario-piotrowicz can you check out @MattIPv4's review? Once resolved, this should be good to land

@dario-piotrowicz
Copy link
Member Author

@dario-piotrowicz can you check out @MattIPv4's review? Once resolved, this should be good to land

Yes, of course, sorry I wasn't around the last few days and couldn't check this, I'm on it now 👍

Co-authored-by: Matt Cowley <[email protected]>
Signed-off-by: Dario Piotrowicz <[email protected]>
@dario-piotrowicz
Copy link
Member Author

Thanks all! 😄

@dario-piotrowicz dario-piotrowicz added this pull request to the merge queue Oct 14, 2025
Merged via the queue into nodejs:main with commit 5ebca06 Oct 14, 2025
12 checks passed
@dario-piotrowicz dario-piotrowicz deleted the dario/open-next-continuous-deployment branch October 14, 2025 15:30
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.

7 participants