Skip to content

Updating packages#1530

Merged
aaronpowell merged 1 commit intostagedfrom
dependency-updates
Apr 28, 2026
Merged

Updating packages#1530
aaronpowell merged 1 commit intostagedfrom
dependency-updates

Conversation

@aaronpowell
Copy link
Copy Markdown
Contributor

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • I have read and followed the Guidance for submissions involving paid services.
  • My contribution adds a new instruction, prompt, agent, skill, or workflow file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, agent, skill, or workflow with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.
  • I am targeting the staged branch for this pull request.

Description


Type of Contribution

  • New instruction file.
  • New prompt file.
  • New agent file.
  • New plugin.
  • New skill file.
  • New agentic workflow.
  • Update to existing instruction, prompt, agent, plugin, skill, or workflow.
  • Other (please specify):

Additional Notes


By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

Copilot AI review requested due to automatic review settings April 28, 2026 01:56
@aaronpowell aaronpowell merged commit 0c31682 into staged Apr 28, 2026
4 checks passed
@aaronpowell aaronpowell deleted the dependency-updates branch April 28, 2026 01:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the website’s npm dependencies to newer versions (notably Astro, Starlight, Marked, and Shiki) and refreshes the lockfile accordingly.

Changes:

  • Bump astro, @astrojs/starlight, @astrojs/sitemap, marked, and shiki versions in website/package.json.
  • Regenerate website/package-lock.json to capture updated transitive dependencies and engine constraints.
Show a summary per file
File Description
website/package.json Updates direct website dependencies to newer versions.
website/package-lock.json Updates resolved dependency graph, including new Node engine requirements introduced by upgrades.

Copilot's findings

  • Files reviewed: 1/2 changed files
  • Comments generated: 2

Comment thread website/package.json
Comment on lines +21 to +24
"@astrojs/sitemap": "^3.7.2",
"@astrojs/starlight": "^0.38.4",
"astro": "^6.1.9",
"choices.js": "^11.2.2",
Copy link

Copilot AI Apr 28, 2026

Choose a reason for hiding this comment

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

Upgrading to Astro 6.x raises the minimum supported Node.js version (the lockfile shows astro@6.1.9 requires node >=22.12.0). Our CI workflows for the website currently use Node 20 (e.g., .github/workflows/build-website.yml and deploy-website.yml), so builds/installations will fail unless the repo’s Node version is bumped accordingly (and ideally reflected via an engines.node field). If the repo must stay on Node 20, astro should remain on a compatible major version instead.

Copilot uses AI. Check for mistakes.
Comment thread website/package-lock.json
Comment on lines 2133 to 2135
"engines": {
"node": "18.20.8 || ^20.3.0 || >=22.0.0",
"node": ">=22.12.0",
"npm": ">=9.6.5",
Copy link

Copilot AI Apr 28, 2026

Choose a reason for hiding this comment

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

The updated lockfile records astro@6.1.9 with engines.node >=22.12.0. This will conflict with the repo’s current GitHub Actions website workflows that install/build with Node 20, causing CI failures unless the workflows (and any documented/tooling Node version) are updated to >=22.12.0.

Copilot uses AI. Check for mistakes.
fizznix pushed a commit to fizznix/awesome-copilot that referenced this pull request Apr 29, 2026
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