Skip to content

chore(deps): migrate svelte-sitemap from CLI to Vite plugin#208

Merged
josecelano merged 1 commit into
developfrom
migrate-svelte-sitemap-to-vite-plugin
Jun 15, 2026
Merged

chore(deps): migrate svelte-sitemap from CLI to Vite plugin#208
josecelano merged 1 commit into
developfrom
migrate-svelte-sitemap-to-vite-plugin

Conversation

@josecelano

Copy link
Copy Markdown
Member

Summary

Migrate svelte-sitemap from the deprecated CLI postbuild script to the recommended Vite plugin. This eliminates the deprecation warning on every build.

Changes

  • Add svelteSitemap({ domain: 'https://torrust.com/' }) plugin to vite.config.ts
  • Remove postbuild script from package.json

Checklist

  • npm run lint passes
  • npm run check passes
  • npm run build succeeds — sitemap now generated via Method: Vite plugin (no deprecation warning)

Copilot AI review requested due to automatic review settings June 15, 2026 12:26
@josecelano josecelano self-assigned this Jun 15, 2026
@josecelano

Copy link
Copy Markdown
Member Author

ACK c179bb2

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Migrates sitemap generation in this SvelteKit/Vite codebase from the deprecated svelte-sitemap CLI (npm postbuild hook) to the recommended Vite plugin, so sitemap generation happens during vite build without the deprecation warning.

Changes:

  • Register svelteSitemap({ domain: 'https://torrust.com/' }) in vite.config.ts.
  • Remove the postbuild npm lifecycle script that invoked the deprecated CLI.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
vite.config.ts Adds the svelte-sitemap Vite plugin to generate sitemap.xml as part of the Vite build pipeline.
package.json Removes the deprecated postbuild script that ran the svelte-sitemap CLI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@josecelano josecelano merged commit 2408a7e into develop Jun 15, 2026
2 checks passed
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