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

[MRG] Complete .zenodo.json for contributor credit #1028

Merged
merged 11 commits into from Aug 2, 2022

Conversation

sappelhoff
Copy link
Member

@sappelhoff sappelhoff commented Jul 29, 2022

PR Description

closes #1025

Also updates some infrastructure.

For .zenodo.json format, see: https://developers.zenodo.org/#representation

Merge checklist

Maintainer, please confirm the following before merging:

  • All comments resolved
  • This is not your own PR
  • All CIs are happy
  • PR title starts with [MRG]
  • whats_new.rst is updated
  • PR description includes phrase "closes <#issue-number>"

.zenodo.json Outdated Show resolved Hide resolved
.zenodo.json Outdated Show resolved Hide resolved
.zenodo.json Outdated Show resolved Hide resolved
.zenodo.json Outdated Show resolved Hide resolved
.zenodo.json Show resolved Hide resolved
.zenodo.json Show resolved Hide resolved
.zenodo.json Outdated Show resolved Hide resolved
.zenodo.json Outdated Show resolved Hide resolved
.zenodo.json Outdated Show resolved Hide resolved
.zenodo.json Show resolved Hide resolved
.zenodo.json Show resolved Hide resolved
.zenodo.json Show resolved Hide resolved
.zenodo.json Show resolved Hide resolved
.zenodo.json Show resolved Hide resolved
@sappelhoff
Copy link
Member Author

Ah -- I just saw that you started to fill in some values @drammock -- thanks for the help!

@codecov
Copy link

codecov bot commented Jul 29, 2022

Codecov Report

Merging #1028 (ed727db) into main (61bf111) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1028   +/-   ##
=======================================
  Coverage   95.19%   95.19%           
=======================================
  Files          25       25           
  Lines        3787     3787           
=======================================
  Hits         3605     3605           
  Misses        182      182           

Help us with your feedback. Take ten seconds to tell us how you rate us.

@sappelhoff sappelhoff added this to the 0.11 milestone Jul 29, 2022
.zenodo.json Outdated Show resolved Hide resolved
.zenodo.json Outdated Show resolved Hide resolved
@mscheltienne
Copy link
Member

{
 "affiliation": "Human Neuroscience Platform, Fondation Campus Biotech Geneva, Geneva, Switzerland"
 "name": "Mathieu Scheltienne",
 "orcid": "0000-0001-8316-7436"
}

@sappelhoff Here it is, thank you!

.zenodo.json Outdated Show resolved Hide resolved
@sappelhoff sappelhoff mentioned this pull request Aug 1, 2022
Copy link
Member Author

@sappelhoff sappelhoff left a comment

Choose a reason for hiding this comment

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

I filled some affiliation information with best guesses based on recent papers I found by the authors. If several affiliations were provided, I always picked the first.

For those where I couldn't find details I left just the name or name+orcid.

We can always update these details in the future, so if you are seeing this later --- just comment here and tag me, or open a new issue / pr.

EDIT: Note also that I intend to migrate this .zenodo.json file to a CITATION.cff file, see: #460 (comment)

@sappelhoff sappelhoff changed the title Complete .zenodo.json for contributor credit [MRG] Complete .zenodo.json for contributor credit Aug 2, 2022
@sappelhoff sappelhoff enabled auto-merge (squash) August 2, 2022 09:00
@sappelhoff sappelhoff merged commit abea1a6 into mne-tools:main Aug 2, 2022
@sappelhoff sappelhoff deleted the zenodo branch August 2, 2022 09:59
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.

complete .zenodo.json metadata
9 participants