Skip to content

Fix: DOI badge#592

Merged
brian-rose merged 1 commit intoProjectPythia:mainfrom
agoose77:patch-1
Feb 16, 2026
Merged

Fix: DOI badge#592
brian-rose merged 1 commit intoProjectPythia:mainfrom
agoose77:patch-1

Conversation

@agoose77
Copy link
Copy Markdown
Contributor

I noticed that the DOI badge wasn't rendering. I used the "link to all versions" DOI and followed the Zenodo scheme for building a badge URI to make this PR.

@agoose77 agoose77 requested a review from a team as a code owner January 20, 2026 12:00
@agoose77 agoose77 requested review from clyne and dcamron and removed request for a team January 20, 2026 12:00
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 20, 2026

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: dad575e
✅ Deployment Preview URL: In Progress

@clyne clyne requested review from jukent and removed request for clyne January 20, 2026 15:42
@brian-rose
Copy link
Copy Markdown
Member

Related to #553

@kafitzgerald
Copy link
Copy Markdown
Contributor

Looks like we're getting an old version of nodejs in the environment that's causing the failures:

It's been happening in the nightly builds as well since Jan 10: https://github.com/ProjectPythia/pythia-foundations/actions/workflows/nightly-build.yaml

Still trying to track down why this is the case / what changed.

@brian-rose
Copy link
Copy Markdown
Member

We resolved the build with #594 (though we don't really understand the underlying problem with different node versions being installed).

Merging latest main into the PR should get it building.

Copy link
Copy Markdown
Member

@brian-rose brian-rose left a comment

Choose a reason for hiding this comment

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

Will merge this as the build failure was unrelated

@brian-rose brian-rose merged commit 1272e87 into ProjectPythia:main Feb 16, 2026
1 of 3 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.

3 participants