Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Zenodo badge #624

Merged
merged 1 commit into from
Feb 7, 2025
Merged

Update Zenodo badge #624

merged 1 commit into from
Feb 7, 2025

Conversation

wheeheee
Copy link
Member

@wheeheee wheeheee commented Feb 6, 2025

The webhook triggered 3 different deliveries on the most recent release, failed twice after the first. But it's on zenodo anyhow so it doesn't matter.

This Zenodo badge will point to the most recent release as suggested previously so no further need to update this badge afterwards.

The webhook triggered 3 different deliveries on the most recent release, failed twice after the first. But it's on zenodo anyhow so it doesn't matter.

This Zenodo badge will point to the most recent release as suggested previously so no further need to update this badge afterwards.
@wheeheee wheeheee enabled auto-merge (squash) February 6, 2025 16:20
Copy link

codecov bot commented Feb 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.13%. Comparing base (2257010) to head (aade01b).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #624   +/-   ##
=======================================
  Coverage   98.13%   98.13%           
=======================================
  Files          19       19           
  Lines        3277     3277           
=======================================
  Hits         3216     3216           
  Misses         61       61           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@martinholters
Copy link
Member

The webhook triggered 3 different deliveries on the most recent release, failed twice after the first. But it's on zenodo anyhow so it doesn't matter.

Creating a release apparently triggers "released", "published", and "created" webhooks. Zenodo correctly acts on the first one it receives and then rejects the other two because "The release has already been received." So just a cosmetic problem.

Copy link
Member

@martinholters martinholters left a comment

Choose a reason for hiding this comment

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

I meant to do this, but forgot... Thanks for fixing this.

@wheeheee wheeheee merged commit 4a88779 into master Feb 7, 2025
11 checks passed
@wheeheee wheeheee deleted the zen_badge branch February 7, 2025 06:58
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