Skip to content

Conversation

@davidsneighbour
Copy link
Contributor

50 members looks better than 49.

Signed-off-by: Patrick Kollitsch <83281+davidsneighbour@users.noreply.github.com>
Copilot AI review requested due to automatic review settings December 29, 2025 06:12
Signed-off-by: Patrick Kollitsch <83281+davidsneighbour@users.noreply.github.com>
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 pull request adds a new member entry for kollitsch.dev to reach 50 members (up from 49). The addition follows the established member file structure with frontmatter containing name, host, feed, SSG, and type information.

  • Adds Patrick Kollitsch's website as a new community member
  • Uses Astro as the SSG
  • Describes the site as a weblog covering personal topics, music, and technology

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

Comment on lines +3 to +4
host: kollitsch.dev
feed: kollitsch.dev/rss.xml
Copy link

Copilot AI Dec 29, 2025

Choose a reason for hiding this comment

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

The host and feed fields include the "https://" protocol prefix, which is inconsistent with the majority of other member files. Most member entries in the codebase omit the protocol (see bobmonsour.md:3-4, zachleat.md:3-4, jeremy.md:3-4, helenchong.md:3-4). While a few files like billglover.md include the protocol in the feed field, the host field consistently omits it across all other entries. For consistency with the established pattern, consider removing "https://" from both fields.

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@cjerrington cjerrington merged commit a6180da into cjerrington:main Dec 29, 2025
@cjerrington
Copy link
Owner

Wow, 50 members! Glad to have you here.

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