Skip to content

Addition of citation file - #34

Closed
HendrikBorgelt wants to merge 4 commits into
nfdi-de:mainfrom
HendrikBorgelt:addition-of-citation-file
Closed

Addition of citation file#34
HendrikBorgelt wants to merge 4 commits into
nfdi-de:mainfrom
HendrikBorgelt:addition-of-citation-file

Conversation

@HendrikBorgelt

Copy link
Copy Markdown
Collaborator

Plus i also fixed the Doi badge, which for some reason had thewrong link to the SVG it requires.

Comment thread README.md Outdated

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I am quite puzzled why the doi badge does not get render correctly if i look at the readme in a fork, but is rendered correctly in the main repo. Also the code on nfdi-de:main diverse from the base code for the doi badge which zenodo provides.

@dalito dalito Nov 26, 2025

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I also observed that DOI pages are sometimes not rendered correctly. Currently the one in this repo shows up for me. Maybe the server is just overloaded.

I used exactly the markdown that Zenodo provides.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

should i ommit the change in the doi badge?

@dalito dalito Nov 26, 2025

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yes, please revert. I think it was fine as it was and if it displays correctly or not depends on the external server only.

Comment thread CITATION.cff
given-names: "Hendrik"
orcid: "https://orcid.org/0000−0001−5886−7860"
title: "ChemDCAT-AP: Enabling Semantic Interoperability with a Contextual Extension of DCAT-AP"
version: 0.1.0 rc1

@dalito dalito Nov 26, 2025

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is a version required? If the cff should be for all versions we may drop it. The doi ending in 369 is for all versions. Hmm, there is also the date-released below...

I am kind of undecided what is best,

  • general and approximate which needs no updates or
  • specific and precise which needs continuous updates

@StroemPhi

Copy link
Copy Markdown
Member

What speaks against merging this?

@dalito

dalito commented Dec 4, 2025

Copy link
Copy Markdown
Collaborator

I thought Hendrik would revert the change in README.md

@HendrikBorgelt
HendrikBorgelt deleted the addition-of-citation-file branch May 18, 2026 10:54
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