Skip to content

Conversation

@kgryte
Copy link
Member

@kgryte kgryte commented Dec 19, 2025

Checklist

Please ensure the following tasks are completed before submitting this pull request.

  • Read and understand the Code of Conduct.
  • Read and understood the licensing terms.
  • Searched for existing issues and pull requests before submitting this pull request.
  • Filed an issue (or an issue already existed) prior to submitting this pull request.
  • Rebased onto latest master.
  • Submitted against master branch.

Description

What is the purpose of this pull request?

This pull request:

  • moves the stdlib installation which is used for building API reference documentation out of the node_modules tree.
  • more cleanly leverages Make to avoid repeated installation when a stdlib installation already exists.
  • renames environment variables.
  • refactors scripts to no longer resolve a stdlib installation from the node_modules tree.
  • removes the need for setting the NODE_PATH env variable when building the app.

Related Issues

Does this pull request have any related issues?

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

I performed various spot checks, such as building the TypeScript documentation, versions file, and HTML fragments, and everything seemed to work as intended.

I did spot some new files when generating the TS documentation, and I am not sure if this is due to upstream changes in typedoc. This should be confirmed when we go to publish the next version of the website.


@stdlib-js/reviewers

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