Skip to content

Use the NPM dependencies from the gha/v0 branch#507

Draft
ppkarwasz wants to merge 2 commits into
mainfrom
feat/main/npm-central-deps
Draft

Use the NPM dependencies from the gha/v0 branch#507
ppkarwasz wants to merge 2 commits into
mainfrom
feat/main/npm-central-deps

Conversation

@ppkarwasz

Copy link
Copy Markdown
Member

Companion of #506, which centralizes the NPM dependencies of all Apache Logging websites on the gha/v0 branch.

  • The committed package.json shrinks to a single Git dependency on gha/v0; package-lock.json is no longer committed.
  • Website deployments for staging and production pass npm-dependencies-ref: gha/v0 to deploy-site-reusable, so they build with the exact dependency tree from that branch. Release website staging keeps using the committed package.json.
  • Dependabot NPM updates now target the gha/v0 branch only.
  • The documentation examples show the new input.

Note

Requires #506 to be merged first, since the Git dependency and the download step point at files that only exist on gha/v0 after that merge.

With these two PRs, our live staging and production website will always use the latest dependencies from the gha/v0 branch.

The only gap remaining is how to handle site generation from the source archive. In practice none of our websites is build from the source archive, but users should be able to generate some documentation from those sources too.

Replace the NPM dependencies with a single Git dependency on the
`gha/v0` branch, where Dependabot keeps the dependencies of all Apache
Logging websites up-to-date at once.

Website deployments pass the new `npm-dependencies-ref` input to
`deploy-site-reusable`, which downloads `package.json` and
`package-lock.json` from that branch before building the website.
Release website staging keeps using the `package.json` committed in
the repository.

Making website builds from released source archives reproducible will
be addressed separately.

Assisted-By: Claude Fable 5 <noreply@anthropic.com>
Prove that the centralized NPM dependencies work before #506 is merged.
Revert this commit once the manifests are available on `gha/v0`.

Assisted-By: Claude Fable 5 <noreply@anthropic.com>
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