-
Notifications
You must be signed in to change notification settings - Fork 1
Fix/vite preprocess import (Security), update colors to Vermilion, trivial fixes #8
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
Merged
jdorfman
merged 11 commits into
sourcegraph:main
from
jdorfman:fix/vite-preprocess-import
Jan 9, 2026
Merged
Fix/vite preprocess import (Security), update colors to Vermilion, trivial fixes #8
jdorfman
merged 11 commits into
sourcegraph:main
from
jdorfman:fix/vite-preprocess-import
Jan 9, 2026
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Change import from '@sveltejs/kit/vite' to '@sveltejs/vite-plugin-svelte' - Resolves SyntaxError: module does not provide export named 'vitePreprocess' Amp-Thread-ID: https://ampcode.com/threads/T-c88bfafb-d4c0-454f-a2b1-b429beda9af5 Co-authored-by: Amp <[email protected]>
- Upgrade @sveltejs/adapter-static from ^2.0.1 to ^3.0.10 - Upgrade @sveltejs/kit from ^1.5.0 to ^2.49.2 - Upgrade svelte from ^3.54.0 to ^5.45.8 - Upgrade vite from ^4.0.0 to ^6.4.1 - Upgrade vitest from ^0.31.0 to ^4.0.15
- Change primary and secondary color values to new vermilion shades - Add new vermilion color variants for enhanced theming
- Update "Github" to "GitHub" for consistency across the codebase - Ensure proper naming in search examples for better clarity
…ind.config.cjs - Add new color variables for primary hover and dark text - Update button styles for primary and secondary buttons to improve hover effects and text colors - Ensure consistent theming across light and dark modes
- Change Node.js version from 16 to 20 in the test workflow - Ensures compatibility with the latest features and improvements
- Add .claude/settings.local.json to .gitignore to prevent local configuration from being tracked
- Change Node.js version from 22 to 18.13 in the test workflow - Ensures compatibility with existing project dependencies
- Change Node.js version from 18.13 to 22.0.0 in the test workflow - Ensures compatibility with the latest features and improvements
trly
suggested changes
Jan 8, 2026
trly
left a comment
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.
Other than the GH action versions, everything else looks 👍🏻
- Update actions/checkout from v4 to v6 - Update actions/setup-node from v4 to v6 - Ensures compatibility with the latest GitHub Actions features
trly
approved these changes
Jan 8, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Before:

After:
