Skip to content

add meetings#112

Merged
addiebart merged 2 commits intomasterfrom
dev
Mar 5, 2026
Merged

add meetings#112
addiebart merged 2 commits intomasterfrom
dev

Conversation

@addiebart
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings March 5, 2026 23:31
Copy link

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

Adds/updates meeting data used by the site’s calendar/next-meeting UI and the archive listing.

Changes:

  • Updated the March 4, 2026 meeting subtitle (speaker name change) and added a new March 12, 2026 upcoming meeting entry.
  • Added a new Spring 25-26 meeting entry to the archive, including a slides link.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/data/meetings.ts Updates upcoming meeting metadata (subtitle) and adds a new upcoming meeting.
src/data/archive.ts Adds a new archived meeting item (date, description, tags, slides link).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +158 to +159
title: "ACM x WIC: Leetcode and Art Night",
subtitle: "ACM joins Women in Computing for a Leetcode and art combo meeting!",
Copy link

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Leetcode" is typically styled as "LeetCode" (brand capitalization). Consider updating the title/subtitle to match the official spelling for consistency.

Suggested change
title: "ACM x WIC: Leetcode and Art Night",
subtitle: "ACM joins Women in Computing for a Leetcode and art combo meeting!",
title: "ACM x WIC: LeetCode and Art Night",
subtitle: "ACM joins Women in Computing for a LeetCode and art combo meeting!",

Copilot uses AI. Check for mistakes.
]
},
{
date: new Date("03/04/2026"),
Copy link

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This entry uses a non-ISO date string ("03/04/2026"), but the file header comment says parsed date strings are ISO (YYYY-MM-DD). To avoid ambiguity in JS date parsing and keep docs accurate, consider switching this (and ideally the file) to ISO format or updating the header comment to match the actual accepted formats.

Suggested change
date: new Date("03/04/2026"),
date: new Date("2026-03-04"),

Copilot uses AI. Check for mistakes.
@addiebart addiebart merged commit de18082 into master Mar 5, 2026
4 checks passed
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