Skip to content

Conversation

@redadmin-k
Copy link

  • Per-release CSV sources (8/9/10, both x86_64 and aarch64) are introduced:
    • data/aws_amis_8.csv
    • data/aws_amis_9.csv
    • data/aws_amis_10.csv
  • Prebuild script auto-merges these into the legacy flat CSV and Markdown:
    • docs/.vuepress/public/ci-data/aws_amis.csv
    • docs/cloud/AWS_AMIS.md
  • Artifacts' paths are unchanged.
  • After this, update only the per-release CSVs; all outputs regenerate automatically.
  • Site build with VuePress is confirmed working.

@redadmin-k
Copy link
Author

redadmin-k commented Sep 12, 2025

Thanks for the review — good catch.

I re-added vuepress and dayjs to package.json and pushed an update to this branch. I ran npm install and confirmed the site builds locally (npm run build). The generated artifacts are unchanged.

Please let me know if you prefer a different placement (e.g. move dayjs to devDependencies).

What
Re-added the build dependencies to package.json so the site can be reproduced by reviewers.

Changes
Restored vuepress to devDependencies.
Restored dayjs to dependencies.
Updated package-lock.json via npm install.
Verification
Ran npm install.
Ran npm run build and confirmed the site builds successfully (output in docs/.vuepress/dist).
Generated artifacts are unchanged.
Notes
I kept vuepress in devDependencies and dayjs in dependencies; happy to move dayjs to devDependencies if preferred.
There were some npm audit warnings during install; they are unrelated to this PR and can be addressed separately.
Signed-off-by: Akiyoshi Kurita [email protected]

@redadmin-k Reply...
@redadmin-k
build: restore vuepress and dayjs to package.json so site build is re…
905882e
Merge info
Review required
At least 1 approving review is required by reviewers with write access.

1 workflow awaiting approval
This workflow requires approval from a maintainer. Learn more about approving workflows.

Merging is blocked
At least 1 approving review is required by reviewers with write ac

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