Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: use defined urls for distribution and api docs #5494

Merged
merged 1 commit into from
Jul 10, 2023
Merged

chore: use defined urls for distribution and api docs #5494

merged 1 commit into from
Jul 10, 2023

Conversation

ovflowd
Copy link
Member

@ovflowd ovflowd commented Jul 10, 2023

Description

This PR introduces Environment-defined constants that can be used to define custom URLs that should be used for the Node.js Distribution CDN (nodejs.org/dist) and Node.js API Docs URL (nodejs.org/docs).

Note that these changes only affect JavaScript and TypeScript files, not any Markdown pages. Which is fine as in the future, these pages will get replaced.

Also, these environment variables should at best be used for testing.

Related Issues

Relates to nodejs/build#3366

@ovflowd ovflowd requested a review from a team as a code owner July 10, 2023 09:21
@vercel
Copy link

vercel bot commented Jul 10, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2023 9:28am
nodejs-org-stories ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2023 9:28am

@ovflowd ovflowd added fast-track Fast Tracking PRs infrastructure Issues/PRs related to the Repository Infra labels Jul 10, 2023
@ovflowd
Copy link
Member Author

ovflowd commented Jul 10, 2023

Requesting Fast-Tracking cc @nodejs/website

@vercel vercel bot temporarily deployed to Preview – nodejs-org-stories July 10, 2023 09:25 Inactive
@vercel vercel bot temporarily deployed to Preview – nodejs-org July 10, 2023 09:28 Inactive
@ovflowd ovflowd merged commit 4346870 into nodejs:main Jul 10, 2023
@ovflowd ovflowd deleted the chore/dist-docs-urls branch July 10, 2023 11:40
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 10, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 10, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 10, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 10, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 10, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 10, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 10, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 10, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 10, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 12, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 12, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 12, 2023
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fast-track Fast Tracking PRs infrastructure Issues/PRs related to the Repository Infra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants