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

authorship, citation, and attribution policies #419

Merged
merged 14 commits into from
Apr 12, 2023
Merged

Conversation

JessicaS11
Copy link
Member

@JessicaS11 JessicaS11 commented Mar 28, 2023

Our recent publication of all releases on Zenodo required we revisit our outdated authorship/attribution guidelines and update them to our current practices. This PR aims to update, standardize, and simplify our practices to universally recognize the "icepyx Developers" as the first author (and only author on Zenodo) of our software package.

Partially address #414.

EDIT: I would be interested in finding a way to highlight the elusively defined "significant" contributions, as well as maintainers... would be happy to add that to this PR or in a future one.

@github-actions
Copy link

github-actions bot commented Mar 28, 2023

Binder 👈 Launch a binder notebook on this branch for commit 31c0f41

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit 61d589f

Binder 👈 Launch a binder notebook on this branch for commit 910648b

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

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

Binder 👈 Launch a binder notebook on this branch for commit 6198e73

Binder 👈 Launch a binder notebook on this branch for commit 976730e

Binder 👈 Launch a binder notebook on this branch for commit 4ce5f1b

Binder 👈 Launch a binder notebook on this branch for commit 0e9a768

@codecov-commenter
Copy link

codecov-commenter commented Mar 28, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (d9a9d94) 59.09% compared to head (976730e) 59.09%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files
@@             Coverage Diff              @@
##           development     #419   +/-   ##
============================================
  Coverage        59.09%   59.09%           
============================================
  Files               30       30           
  Lines             2264     2264           
  Branches           488      488           
============================================
  Hits              1338     1338           
  Misses             843      843           
  Partials            83       83           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@JessicaS11 JessicaS11 marked this pull request as ready for review April 5, 2023 18:44
@JessicaS11 JessicaS11 linked an issue Apr 5, 2023 that may be closed by this pull request
3 tasks
Copy link
Contributor

@kelseybisson kelseybisson left a comment

Choose a reason for hiding this comment

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

I like the changes and I think the language used is good. I also think we could add a consideration of 'J. Scheick and icepyx developers' to the mix so Jessica's contributions can be emphasized rather than having all contributions remain anonymous.

Copy link
Member

@weiji14 weiji14 left a comment

Choose a reason for hiding this comment

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

Just two suggestions on the CITATION.cff file, otherwise ok!

doc/CITATION.cff Outdated Show resolved Hide resolved
doc/CITATION.cff Outdated Show resolved Hide resolved
CONTRIBUTORS.rst Outdated Show resolved Hide resolved
CONTRIBUTORS.rst Outdated Show resolved Hide resolved
@JessicaS11 JessicaS11 merged commit e3cf86b into development Apr 12, 2023
@JessicaS11 JessicaS11 deleted the authors branch April 12, 2023 18:07
@JessicaS11 JessicaS11 linked an issue Apr 17, 2023 that may be closed by this pull request
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.

improve Zenodo releases update attribution guidelines
4 participants