Conversation
There was a problem hiding this comment.
Pull request overview
Adds new meeting/archive entries and tweaks the archive page layout for better mobile sizing.
Changes:
- Adjust archive page content width to be more mobile-friendly (
w-4/5withmd:w-2/3). - Add a new upcoming general meeting entry (Mar 4, 2026).
- Add new archive items for a general meeting and a BugSmashers session; fix a BugSmashers title typo.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
src/pages/archive.astro |
Makes the archive listing container more responsive on smaller screens. |
src/data/meetings.ts |
Adds an upcoming meeting entry to the general meetings list. |
src/data/bugsmashers.ts |
Fixes a spelling error in a BugSmashers meeting title. |
src/data/archive.ts |
Adds new archive records for a meeting and a BugSmashers event (with slides links). |
.astro/settings.json |
Updates a tooling timestamp value (likely editor state). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| { | ||
| "_variables": { | ||
| "lastUpdateCheck": 1770941154036 | ||
| "lastUpdateCheck": 1772162251231 |
There was a problem hiding this comment.
This looks like editor/tooling state (last update check timestamp) rather than app configuration, and tends to cause noisy diffs and merge conflicts. Suggest reverting this change and adding .astro/settings.json to .gitignore (or otherwise keeping it out of PRs) if it isn’t required at runtime.
| "lastUpdateCheck": 1772162251231 | |
| "lastUpdateCheck": 0 |
| date: "February 26, 2026", | ||
| title: "BugSmashers Takeover: Game Night!", | ||
| subtitle: "The BugSmashers take over this week's general meeting to bring out some games! Be there!", | ||
| }, | ||
| { |
There was a problem hiding this comment.
The "BugSmashers Takeover: Game Night!" entry is dated Feb 26, 2026 (a Thursday), but general meetings appear to be Wednesdays (and the archive entry for this meeting is 02/25/2026). This likely makes the calendar/NextMeeting display incorrect; update this date to the correct Wednesday (or otherwise ensure it matches the archive and actual schedule).
| date: new Date("02/25/2026"), | ||
| name: "BugSmashers Takeover: Game Night", |
There was a problem hiding this comment.
This file’s header comment states parsed date strings are ISO (YYYY-MM-DD), but this new entry uses an MM/DD/YYYY string. Since non-ISO date-string parsing is implementation-dependent in JavaScript, prefer an ISO string (e.g., YYYY-MM-DD) or the numeric Date constructor for reliability and to match the documented format.
| date: new Date('02/26/2026'), | ||
| description: 'BugSmashers discuss different parts of a computer and configure a system with PCPartPicker.', |
There was a problem hiding this comment.
This new BugSmashers archive item uses a non-ISO date string ('02/26/2026'). JavaScript Date parsing for non-ISO formats is not guaranteed across runtimes; use an ISO date string (YYYY-MM-DD) or Date(year, monthIndex, day) to avoid inconsistent parsing.
No description provided.