Conversation
There was a problem hiding this comment.
Pull request overview
Adds new Spring 2026 archive entries to the site’s archive data (ALL_ARCHIVE_ITEMS) so they appear on the archive page.
Changes:
- Added a new Spring 2026 “Meetings” entry for “Alarm.com Speaker” (03/25/2026).
- Added a new Spring 2026 “Bugsmashers” entry for “BugSmashers: Quantum Computing” (03/26/2026).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| date: new Date("03/25/2026"), | ||
| name: "Alarm.com Speaker", | ||
| description: "Jeff Kissick of Alarm.com discussed his role in industry and Alarm.com's internship program.", |
There was a problem hiding this comment.
new Date("03/25/2026") relies on non-ISO date parsing, which is implementation-dependent in JavaScript. The file header also states date strings are ISO (YYYY-MM-DD); consider switching this (and similar new entries) to an ISO string (e.g., "2026-03-25") or a numeric constructor to avoid cross-runtime parsing issues.
| semester: 'Spring', | ||
| date: new Date('03/26/2026'), | ||
| description: 'BugSmashers introduce quantum computing concepts and discuss their effects on the discipline.', |
There was a problem hiding this comment.
new Date('03/26/2026') uses a non-ISO date string, which is parsed in an implementation-dependent way by JavaScript. To align with the file header comment and avoid parsing differences across environments/timezones, prefer an ISO string ("2026-03-26") or new Date(2026, 2, 26).
| { | ||
| name: 'BugSmashers: Quantum Computing', | ||
| semester: 'Spring', | ||
| date: new Date('03/26/2026'), | ||
| description: 'BugSmashers introduce quantum computing concepts and discuss their effects on the discipline.', | ||
| tags: ["Talks", "Workshops"], | ||
| entryType: "Bugsmashers", | ||
| year: "25-26", |
There was a problem hiding this comment.
The PR title suggests this change is only about adding meetings, but this hunk also adds a new Bugsmashers archive item. Either update the title/description to reflect both additions, or split the BugSmashers update into a separate PR for clearer history.
No description provided.