Skip to content

Add entries to citation file#1254

Merged
danielfromearth merged 2 commits intomainfrom
update-citation
Mar 10, 2026
Merged

Add entries to citation file#1254
danielfromearth merged 2 commits intomainfrom
update-citation

Conversation

@danielfromearth
Copy link
Contributor

@danielfromearth danielfromearth commented Mar 6, 2026

This PR is intended to update the CITATION.cff with new entries. This will also be reflected in updates to the draft manuscript for the Journal of Open Source Software (JOSS) in #1249.

To add:

Who else should be added – are there others to include here?

@github-actions
Copy link

github-actions bot commented Mar 6, 2026

Binder 👈 Launch a binder notebook on this branch for commit 0866353

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit f83df26

@mfisher87 mfisher87 marked this pull request as ready for review March 6, 2026 20:47
@mfisher87
Copy link
Member

mfisher87 commented Mar 6, 2026

@chuckwondo do you have an ORCiD?

@chuckwondo
Copy link
Contributor

@chuckwondo do you have an ORCiD?

No I don't, so my github link is fine. Thanks!

@danielfromearth
Copy link
Contributor Author

danielfromearth commented Mar 10, 2026

@mfisher87, have you seen this "Check Links" job failing a bunch? Ideas on how to address?

@mfisher87
Copy link
Member

Yeah. This is a tradeoff I'm not sure is worth it anymore, but I do see the job failure in this PR has detected a broken link:

https://www.gnu.org/software/inetutils/manual/html_node/The-_002enetrc-file.html

We should fix that for sure and we wouldn't have noticed it without Check Links.

But then there's also failures because e.g. GitHub has rate limits (HTTP 429). And I think the solution to that is updating the ignore list to add GitHub links, which reduces the value of the check links job. 🤷

@mfisher87
Copy link
Member

For this PR I'd say ignore it and merge. Do you have the ability to bypass and merge?

@danielfromearth danielfromearth merged commit e110dc3 into main Mar 10, 2026
7 of 11 checks passed
@danielfromearth danielfromearth deleted the update-citation branch March 10, 2026 18:41
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.

4 participants