Skip to content

Conversation

@tanhauhau
Copy link
Collaborator

@tanhauhau tanhauhau commented Nov 16, 2025

  • Upgrade dependencies: Astro, Svelte
  • Upgrade linter / formatter to Biome
  • Add Github CI pipeline to check linting/formatting

@netlify
Copy link

netlify bot commented Nov 16, 2025

Deploy Preview for geekcampsg ready!

Name Link
🔨 Latest commit 6cb3cad
🔍 Latest deploy log https://app.netlify.com/projects/geekcampsg/deploys/6919dd460f384f0008de0f14
😎 Deploy Preview https://deploy-preview-298--geekcampsg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

Copilot AI left a 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.

Copy link

Copilot AI commented Nov 16, 2025

@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.

@tanhauhau tanhauhau requested a review from Copilot November 16, 2025 14:19
Copilot finished reviewing on behalf of tanhauhau November 16, 2025 14:20
Copy link
Contributor

Copilot AI left a 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.

Copy link

Copilot AI commented Nov 16, 2025

@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.

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