Skip to content

chore(scripts): stop generating content for archived versions - #4671

Merged
thetaPC merged 3 commits into
mainfrom
archived-scripts
Aug 27, 2026
Merged

chore(scripts): stop generating content for archived versions#4671
thetaPC merged 3 commits into
mainfrom
archived-scripts

Conversation

@thetaPC

@thetaPC thetaPC commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Issue URL: N/A

What is the current behavior?

scripts/native.mjs and scripts/cli.mjs write generated content to v6 and v7 on every build. Both are archived and served from frozen Vercel deployments, so that's 140 files per build that nothing reads.

The archiving checklist in CONTRIBUTING doesn't mention the scripts, so the lists go stale on each release.

What is the new behavior?

  • Both scripts now write only to docs/ (current) and versioned_docs/version-v8 (the one older version still in versions.json)
  • Added step 9 to the CONTRIBUTING archiving checklist covering both scripts

Does this introduce a breaking change?

  • Yes
  • No

Other information

The generated files are git-ignored, so nothing was committed and nothing is removed from the repo. The frozen deployments already contain their own baked copies.

Verified by backdating the v6 and v7 output, then re-running both scripts: those directories stayed untouched while docs/ and version-v8 were rewritten.

@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
ionic-docs Ready Ready Preview Aug 27, 2026 10:52pm

Request Review

@thetaPC
thetaPC marked this pull request as ready for review August 27, 2026 15:25
@thetaPC
thetaPC requested a review from a team as a code owner August 27, 2026 15:25
@thetaPC
thetaPC requested a review from ShaneK August 27, 2026 15:25
Comment thread CONTRIBUTING.md Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we also mention to add the archiving version back to scripts if it was removed? These steps kind of cover both archiving an existing version and an already removed version.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@brandyscarney brandyscarney left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, just noted one place we could add a note.

@thetaPC
thetaPC merged commit ab9d770 into main Aug 27, 2026
4 checks passed
@thetaPC
thetaPC deleted the archived-scripts branch August 27, 2026 23:13
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