-
Notifications
You must be signed in to change notification settings - Fork 28
Remove histHub from entries, update JS dependencies, and migrate to pnpm #293
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
Co-authored-by: maehr <[email protected]>
Deploying awesome-digital-history with
|
| Latest commit: |
aec2f26
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4457cf1d.awesome-digital-history.pages.dev |
| Branch Preview URL: | https://copilot-update-dependencies.awesome-digital-history.pages.dev |
Co-authored-by: maehr <[email protected]>
|
@copilot update JS dependencies |
Co-authored-by: maehr <[email protected]>
Co-authored-by: maehr <[email protected]>
JS dependencies have been updated to their latest compatible versions. Updated packages include @playwright/test (1.57.0), @sveltejs/kit (2.49.2), tailwindcss (3.4.19), typescript (5.9.3), vite (5.4.21), and others. All linting and tests pass. Changes committed in 0d00536 and 46d0f35. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot use pnpm |
Co-authored-by: maehr <[email protected]>
Switched to pnpm package manager. Removed package-lock.json, updated pnpm-lock.yaml with the latest dependencies, and changed the prepare script to use Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request removes the histHub platform from the collection, updates JavaScript dependencies to their latest compatible patch versions, and migrates the project from npm to pnpm as the package manager.
- Removed histHub entry from the data source per issue #292
- Updated 9 JavaScript dependencies to their latest compatible versions (patch updates only)
- Fixed the packageManager field format and updated the prepare script to use pnpm commands
Reviewed changes
Copilot reviewed 3 out of 5 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/lib/data/entries.json | Removed histHub entry completely |
| package.json | Updated 9 dev dependencies to latest patch versions, corrected packageManager field format from "[email protected]" to "[email protected]", and updated prepare script to use pnpm instead of npm |
| pnpm-lock.yaml | Updated lock file to reflect new dependency versions and their transitive dependencies |
| README.md | Auto-generated file reflecting the removal of histHub from the entries list |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Pull Request
Remove
histHubfromsrc/lib/data/entries.json, update JavaScript dependencies to their latest compatible versions, and migrate the project to use pnpm as the package manager.Short pitch
The histHub platform at histhub.ch is being removed from the collection per issue #292. Additionally, JavaScript dependencies have been updated to their latest compatible versions within the same major versions to ensure the project stays current with security patches and bug fixes. The project has also been migrated from npm to pnpm to align with the packageManager specification in package.json.
Changes made:
pnpm runinstead ofnpm runChecklist
Please ensure that you have completed the following tasks:
npm run prepareto sort the entries insrc/lib/data/entries.jsonalphabetically and to generate theREADME.mdfile.npm run formatto format the repository code.npm run awesome-lintto ensure that theREADME.mdfile is formatted correctly.Criteria for accepting a pull request
Contributors, please ensure that:
Maintainers, please ensure that:
Updating your PR
If the maintainers notice anything that needs to be changed, they will ask you to edit your PR before merging it. Please do not open a new PR, just edit the existing one. If you're not sure how to do that, here is a guide on the different ways you can update your PR.
Appendix: running lint tests
To run tests locally using pnpm, you need to install the dependencies first:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.