-
Notifications
You must be signed in to change notification settings - Fork 11
feat: upgrade dependencies #298
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
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for geekcampsg ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
5575350 to
8eae971
Compare
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 PR modernizes the GeekcampSG website by upgrading core dependencies and tooling:
- Upgrades Astro from v3 to v5 and Svelte from v3 to v5
- Replaces Prettier/ESLint with Biome for unified formatting and linting
- Adds GitHub Actions CI pipeline for automated quality checks
- Migrates Svelte components to the new v5 syntax with runes ($props, $state, $derived, $effect)
Reviewed Changes
Copilot reviewed 52 out of 55 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated dependencies to Astro v5 and Svelte v5, replaced Prettier with Biome, added typecheck script |
| biome.json | New Biome configuration for formatting and linting with project-specific rules |
| .github/workflows/ci.yml | New CI pipeline that runs Biome checks, typechecking, and build verification |
| svelte.config.js | Added Svelte configuration with Astro's Vite preprocessor |
| src/pages/_/Speakers/Speakers.svelte | Migrated to Svelte 5 runes syntax with proper TypeScript interfaces |
| src/pages/_/Hero/Game/index.svelte | Converted from svelte-knobby to writable stores, migrated to runes syntax |
| src/data/schedules/*.ts | Reformatted schedule data from JSON-style to proper JavaScript object notation |
| Multiple Astro/Svelte files | Applied consistent formatting via Biome (single quotes, trailing commas) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@tanhauhau I've opened a new pull request, #299, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: tanhauhau <[email protected]>
Fix CSS property typo in Hero component
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
Copilot reviewed 52 out of 55 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@tanhauhau I've opened a new pull request, #300, to work on those changes. Once the pull request is ready, I'll request review from you. |
Uh oh!
There was an error while loading. Please reload this page.