Skip to content

Memberpage created, and linked from the top of the landing page#3

Merged
larsvilhuber merged 14 commits into
masterfrom
memberpage
Jun 11, 2026
Merged

Memberpage created, and linked from the top of the landing page#3
larsvilhuber merged 14 commits into
masterfrom
memberpage

Conversation

@larsvilhuber

Copy link
Copy Markdown
Contributor

Needs everybody to fill in their info.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a members listing to the SSDE Jekyll site and links it from the top navigation, with CI updated to build on pull requests.

Changes:

  • Added a new members.md page that renders a members table from _data/members.csv.
  • Updated the “Who We Are” landing-page section to link to the members page, and added a “Members” item to site navigation.
  • Updated the GitHub Pages workflow to run on pull_request (build-only) and adjusted deploy behavior.

Reviewed changes

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

Show a summary per file
File Description
members.md New members page with a Liquid-generated table from site.data.members.
.github/workflows/jekyll-gh-pages.yml Runs build on PRs and prevents deploy on PR events.
.github/skills/jekyll-ssde-site/SKILL.md Adds repository-specific documentation about the site’s structure/content model.
_posts/2023-01-01-who-we-are.markdown Updates landing-page “Who We Are” section to reference the members page.
_data/members.csv Introduces initial members dataset to power the members table.
_config.yml Adds “Members” to navigation and updates site metadata fields.

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

Comment thread members.md
Comment thread members.md Outdated
Comment thread _config.yml Outdated
Comment thread _config.yml Outdated
@larsvilhuber
larsvilhuber merged commit be534ab into master Jun 11, 2026
2 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