Skip to content

Added .zenodo.json file with instructions. #1924

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

Merged
merged 2 commits into from
Apr 1, 2017

Conversation

chrisgorgo
Copy link
Member

When releasing remember to:

  • Use a consistent release title with the version number
  • Populate the release description with the changelog

@effigies
Copy link
Member

Do we want to hook in Github IDs, as well? Might make it easier to assess whether we've gotten everybody.

@chrisgorgo
Copy link
Member Author

chrisgorgo commented Apr 1, 2017 via email

@codecov-io
Copy link

codecov-io commented Apr 1, 2017

Codecov Report

Merging #1924 into master will increase coverage by 2.36%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1924      +/-   ##
==========================================
+ Coverage   72.54%   74.91%   +2.36%     
==========================================
  Files        1064      873     -191     
  Lines       54178    44615    -9563     
  Branches     7811     7539     -272     
==========================================
- Hits        39306    33422    -5884     
+ Misses      13654    10180    -3474     
+ Partials     1218     1013     -205
Flag Coverage Δ
#smoketests 74.91% <ø> (+2.36%) ⬆️
#unittests 73.06% <ø> (+3.04%) ⬆️
Impacted Files Coverage Δ
...rfaces/slicer/tests/test_auto_SlicerCommandLine.py 81.25% <0%> (-18.75%) ⬇️
nipype/interfaces/dipy/setup.py 7.14% <0%> (-17.86%) ⬇️
nipype/interfaces/camino/convert.py 61.88% <0%> (-9.94%) ⬇️
nipype/interfaces/camino/odf.py 72.72% <0%> (-6%) ⬇️
nipype/workflows/dmri/camino/group_connectivity.py 89.43% <0%> (-4.86%) ⬇️
...semtools/registration/tests/test_auto_BRAINSFit.py 81.81% <0%> (-3.9%) ⬇️
nipype/interfaces/fsl/tests/test_auto_Split.py 82.6% <0%> (-3.11%) ⬇️
nipype/interfaces/fsl/epi.py 62.73% <0%> (-2.81%) ⬇️
nipype/algorithms/tests/test_auto_WarpPoints.py 83.33% <0%> (-2.39%) ⬇️
...ype/interfaces/fsl/tests/test_auto_Level1Design.py 83.33% <0%> (-2.39%) ⬇️
... and 235 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 368e184...ac10302. Read the comment docs.

@effigies
Copy link
Member

effigies commented Apr 1, 2017

Gotcha. I assumed this was your own json construction.

@chrisgorgo
Copy link
Member Author

chrisgorgo commented Apr 1, 2017 via email

@effigies
Copy link
Member

effigies commented Apr 1, 2017

I'd previously done a little digging to find users and affiliations that aren't in the Zenodo so my BibTeX didn't look so ugly. Here are my best estimates as to some missing contributors:

This is an incomplete list based off of GitHub handles in the previous Zenodo releases. I've included these handles, so hopefully they'll see this and alert us if I got anything wrong.

.zenodo.json Outdated
},
{
"affiliation": "Boston University",
"name": "Markiewicz, Christopher",
Copy link
Member

Choose a reason for hiding this comment

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

Would prefer "Markiewicz, Christopher J." for citations.

@chrisgorgo
Copy link
Member Author

Thanks for digging this up. I modified your initial and I will merge it as it is.

Since I do not want to force anyone on the publication or assume what their pen names and affiliations should be I would appreciate if the missing mentioned people send a PRs modifying the .zenodo.json.

@chrisgorgo chrisgorgo merged commit 989a130 into nipy:master Apr 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants