Skip to content

Use Quarto about page format for team bio pages - #76

Open
d-morrison wants to merge 9 commits into
mainfrom
claude/issue-73-20260516-0819
Open

Use Quarto about page format for team bio pages#76
d-morrison wants to merge 9 commits into
mainfrom
claude/issue-73-20260516-0819

Conversation

@d-morrison

@d-morrison d-morrison commented Jun 5, 2026

Copy link
Copy Markdown
Member

Applies the Quarto about page format (https://quarto.org/docs/websites/website-about.html) across all team member bio pages.

Closes #73

Generated with Claude Code

@github-actions @d-morrison
Use Quarto about page format for all team member bio pages
4e771ae 
- Add about.links (email, website, CV) to all student bio pages,
  replacing plain-text contact info in the document body
- Create PI about pages (bio-kaiemjoy.qmd, bio-demorrison.qmd) using
  the Quarto about page template format
- Update team.qmd to reference the new PI about pages instead of
  external-only YAML files

Closes #73

Co-authored-by: Douglas Ezra Morrison <d-morrison@users.noreply.github.com>
@github-actions

github-actions Bot commented Jun 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1-2-g6ad689f

QR code for preview link

🚀 View preview at
https://ucd-serg.github.io/pr-preview/pr-76/

Built to branch gh-pages at 2026-08-01 03:52 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@d-morrison
d-morrison requested a review from kaiemjoy July 31, 2026 07:05
Copilot AI review requested due to automatic review settings July 31, 2026 07:05
Copilot AI reviewed Jul 31, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@d-morrison
d-morrison removed the request for review from kaiemjoy July 31, 2026 07:05
d-morrison and others added 4 commits July 31, 2026 12:00
@d-morrison
Update bio for Ezra Morrison with leadership details
f117038
@d-morrison
Update bio-kaiemjoy.qmd
6d55988
@d-morrison
Merge branch 'main' into claude/issue-73-20260516-0819
d858b88
@claude
Fix broken CV link on Kristen Aiemjoy's bio page
2b9d181 
The about-page link used a bare filename, which Quarto rewrites to a
root-absolute href without the directory prefix, pointing to a file
that doesn't exist there. It also wasn't in the site's copied
resources, since about-page links aren't scanned like body content.
Fixing the href path and declaring the PDF under `resources:` makes
Quarto both link to and copy the file correctly.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0173a6awQ9LNVimJ6ohzdTH9
@d-morrison Claude

d-morrison commented Jul 31, 2026

Copy link
Copy Markdown
Member Author

CI status check: lint-project, Spellcheck, build-deploy, and CodeFactor all pass on the latest commit (2b9d181, which fixes the broken CV link on Kristen's bio page — dropped the local file into resources: and corrected the about.links href path so Quarto both copies and links to the PDF correctly).

The claude-review check is failing, but not due to anything in this diff — the job fails during its own setup, before reviewing any code:

Exchanging OIDC token for app token...
App token exchange failed: 401 Unauthorized - Invalid OIDC token
(retried 3x, all failed)

That's an auth/infrastructure issue with the claude-review workflow's OIDC token exchange (likely an expired or misconfigured credential), not something fixable from this PR. Flagging so it's not mistaken for a merge blocker — probably worth a look at the workflow's token/federation setup separately.

No unresolved review comments otherwise.

Generated by Claude Code

@d-morrison d-morrison mentioned this pull request Aug 1, 2026
Merged
@d-morrison

d-morrison commented Aug 1, 2026

Copy link
Copy Markdown
Member Author

The failing claude-review job here is not caused by anything in this PR.

Run 30677231096 dies at the token-exchange step with 401 Unauthorized - Invalid OIDC token, before Claude reads the diff. Cause: #83 switched the workflow to pull_request_target, and Anthropic's exchange endpoint rejects OIDC tokens minted for that event (anthropics/claude-code-action#713, open upstream). Five of five runs since that merge have failed the same way.

Tracked as #88, fix in #89. Once #89 merges, pushing here (or re-running) will produce a real review.

This was referenced Aug 1, 2026
Merged
Merged
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

use https://quarto.org/docs/websites/website-about.html

3 participants

@d-morrison @claude