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

Format the author list into a table #3373

Merged
merged 1 commit into from
Aug 6, 2024
Merged

Format the author list into a table #3373

merged 1 commit into from
Aug 6, 2024

Conversation

seisman
Copy link
Member

@seisman seisman commented Aug 5, 2024

Just change the list of authors into a table, which looks better IMHO.

Preview

@seisman seisman added documentation Improvements or additions to documentation skip-changelog Skip adding Pull Request to changelog needs review This PR has higher priority and needs review. labels Aug 5, 2024
@seisman seisman added this to the 0.13.0 milestone Aug 5, 2024
Copy link
Member

@yvonnefroehlich yvonnefroehlich left a comment

Choose a reason for hiding this comment

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

Just change the list of authors into a table, which looks better IMHO.

Totally agree! Actually I was thinking about this when fixing the typo in #3313.

@weiji14
Copy link
Member

weiji14 commented Aug 5, 2024

I'm aware that some people on the list have new affiliations, should we update them here, or separately during the release since it will involve a corresponding change to the CITATION.cff file?

@seisman
Copy link
Member Author

seisman commented Aug 5, 2024

I'm aware that some people on the list have new affiliations, should we update them here, or separately during the release since it will involve a corresponding change to the CITATION.cff file?

Definitely in a separate PR since we need to ping them to confirm with the affiliation changes.

@seisman seisman removed the needs review This PR has higher priority and needs review. label Aug 6, 2024
@seisman seisman merged commit dfedf16 into main Aug 6, 2024
7 checks passed
@seisman seisman deleted the author-table branch August 6, 2024 02:03
@seisman
Copy link
Member Author

seisman commented Aug 6, 2024

I'm aware that some people on the list have new affiliations, should we update them here, or separately during the release since it will involve a corresponding change to the CITATION.cff file?

Definitely in a separate PR since we need to ping them to confirm with the affiliation changes.

I'm not sure we want to take the time to update their affiliations. In fact, the affiliations may be outdated but still reflect the affiliations at the time when they contribute. People can still update their affiliations when they come back to contribute to PyGMT again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation skip-changelog Skip adding Pull Request to changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants