chore: update npm dependencies and contributors list#230
Conversation
Updated the following devDependencies: - @sveltejs/kit: ^2.66.0 → ^2.67.0 - @types/node: ^25.6.0 → ^26.0.0 - @typescript-eslint/eslint-plugin: ^8.61.1 → ^8.62.0 - @typescript-eslint/parser: ^8.61.1 → ^8.62.0 - globals: ^17.5.0 → ^17.7.0 - typescript-eslint: ^8.61.1 → ^8.62.0 Closes dependabot PRs torrust#223, torrust#224, torrust#225, torrust#226, torrust#227
There was a problem hiding this comment.
Pull request overview
Updates the project’s npm dependency versions and refreshes the static fallback contributors list used by the website when dynamic contributor loading isn’t available.
Changes:
- Bumped several devDependencies (SvelteKit, TypeScript ESLint packages, globals, and @types/node) and updated the lockfile accordingly.
- Refreshed
defaultContributorsListto include additional contributors from the Torrust GitHub org.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/lib/constants/constants.ts | Extends the static fallback contributors array used for homepage rendering. |
| package.json | Updates devDependency version ranges per the PR summary. |
| package-lock.json | Syncs the lockfile with the updated dependency versions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| { | ||
| html_url: 'mvanhorn', | ||
| avatar_url: 'https://avatars.githubusercontent.com/u/455140?v=4' | ||
| }, |
| "@sveltejs/vite-plugin-svelte": "^7.0.0", | ||
| "@types/dateformat": "^5.0.3", | ||
| "@types/node": "^25.6.0", | ||
| "@types/node": "^26.0.0", |
a2e082d to
6731786
Compare
- Rename html_url field to login in defaultContributorsList (was storing a username, not a URL — misleading field name) - Update the update script to output login instead of html_url - Update Contributors.svelte fallback render to use contributor.login - Bump engines.node from >=20.19.0 to >=26.0.0 to match @types/node v26 - Document fork/archive filtering in the update-contributors skill
c555b21 to
4741716
Compare
|
Both Copilot review comments have been addressed:
|
|
ACK 4741716 |
|
Addressed both Copilot review suggestions: @copilot-pull-request-reviewer — Comment 1 (
The dynamic @copilot-pull-request-reviewer — Comment 2 ( |
Summary
Update npm devDependencies, refresh the contributors list, and address Copilot review feedback.
Changes
Dependency updates
@sveltejs/kit: ^2.66.0 → ^2.67.0@types/node: ^25.6.0 → ^26.0.0@typescript-eslint/eslint-plugin: ^8.61.1 → ^8.62.0@typescript-eslint/parser: ^8.61.1 → ^8.62.0globals: ^17.5.0 → ^17.7.0typescript-eslint: ^8.61.1 → ^8.62.0Contributors
Review feedback addressed
html_url→loginindefaultContributorsList— the field was storing a GitHub username, not a full URL, which was misleadingContributors.sveltefallback render to usecontributor.logininstead ofcontributor.html_urllogininstead ofhtml_urlengines.nodefrom>=20.19.0to>=26.0.0to match the@types/nodev26 upgradeChecklist
npm run lintpassesnpm run checkpasses