Skip to content

Fixing Zenodo badge - #10586

Merged
esc merged 4 commits into
numba:mainfrom
travishathaway:patch-1
May 6, 2026
Merged

Fixing Zenodo badge#10586
esc merged 4 commits into
numba:mainfrom
travishathaway:patch-1

Conversation

@travishathaway

@travishathaway travishathaway commented May 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

The Zenodo badge was broken, so I fixed it. I'm pretty sure this is the right page:

@esc

esc commented May 5, 2026

Copy link
Copy Markdown
Member

@travishathaway thank you for posting this. I did look at the way the README.rst is being rendered on your branch, and it doesn't seem like the badge is showing up:

Screenshot 2026-05-05 at 14 54 13

Is this what you see too?

@travishathaway

travishathaway commented May 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

@esc,

It's rendering correctly only part of the time 🙈. Sometimes it renders correctly, sometimes it returns random HTTP errors like the one shown below.

Bildschirmfoto 2026-05-05 um 15 25 50

It's coming from a GitHub domain, so maybe it's just their service being flaky 🤷. But, I did compare this fix with the current main branch and noticed that at least the badge renders correctly 50% of the time, which is an improvement from 0% 😂

esc
esc reviewed May 5, 2026
View reviewed changes

@esc esc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I searched on the web and one suggestion seems to be that the URL is case-sensitive?? Can you try this?

Comment thread README.rst Outdated
@esc

esc commented May 5, 2026

Copy link
Copy Markdown
Member

@travishathaway ok, I can see it "sometimes" now too, when I refresh the page... must be broken GitHub.

@esc

esc commented May 5, 2026

Copy link
Copy Markdown
Member

@travishathaway can you "cite all versions" instead? The Zendo page says:

Cite all versions? You can cite all versions by using the DOI [10.5281/zenodo.4343230](https://doi.org/10.5281/zenodo.4343230). This DOI represents all versions, and will always resolve to the latest one. [Read more](https://zenodo.org/help/versioning).

@esc esc added 4 - Waiting on author Waiting for author to respond to review and removed 2 - In Progress labels May 5, 2026
esc
esc requested changes May 5, 2026
View reviewed changes

@esc esc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, so please use lowercase doi and then use the link provided to cite all versions.

@esc esc added the skip_release_notes Skip towncrier requirement label May 5, 2026
@travishathaway @esc
Apply suggestion from @esc
9fd0cf0 
Co-authored-by: Emergency Self-Construct <esc@users.noreply.github.com>
travishathaway
travishathaway commented May 5, 2026
View reviewed changes
Comment thread README.rst Outdated
@travishathaway
travishathaway requested a review from esc May 5, 2026 17:03
esc
esc requested changes May 6, 2026
View reviewed changes
Comment thread README.rst Outdated
@travishathaway @esc
Update README.rst
d7a12f3 
Co-authored-by: Emergency Self-Construct <esc@users.noreply.github.com>
esc
esc approved these changes May 6, 2026
View reviewed changes

@esc esc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Image

LGTM

@esc esc added 4 - Waiting on CI Review etc done, waiting for CI to finish and removed 4 - Waiting on author Waiting for author to respond to review labels May 6, 2026
@esc
esc merged commit ba076c1 into numba:main May 6, 2026
26 checks passed
@esc esc added 5 - Ready to merge Review and testing done, is ready to merge and removed 4 - Waiting on CI Review etc done, waiting for CI to finish labels May 6, 2026
@esc

esc commented May 6, 2026

Copy link
Copy Markdown
Member

Thanks, I belive the broken badge is a result of zenodo HTTP being slow and GitHub running into a timeout..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@esc esc esc approved these changes

Assignees

No one assigned

Labels

5 - Ready to merge Review and testing done, is ready to merge skip_release_notes Skip towncrier requirement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@travishathaway @esc @kc611