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

Add Hugo van Kemenade (hugovk) to author list #111

Merged
merged 2 commits into from
Oct 22, 2019

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Oct 22, 2019

For PR 💯

@leouieda leouieda changed the title Update AUTHORS.md Add Hugo van Kemenade (hugovk) to author list Oct 22, 2019
@leouieda
Copy link
Member

@hugovk we've been keeping the list in alphabetical order by last name, so you should probably be first 🙂

@hugovk
Copy link
Member Author

hugovk commented Oct 22, 2019

It depends on which country's surname sort order you use! But this being Python, I guess we can follow Guido's lead :)

https://github.com/python/cpython/blob/dfe726b1ace03f206f45253b93ed7610473ae20f/Misc/ACKS#L1416

Updated!

@leouieda
Copy link
Member

@hugovk Guido knows best :) Thanks!

@leouieda
Copy link
Member

Dutch sort

🤣

@leouieda leouieda merged commit 9a1382d into fatiando:master Oct 22, 2019
@hugovk hugovk deleted the patch-2 branch October 22, 2019 12:22
@hugovk
Copy link
Member Author

hugovk commented Oct 22, 2019

🇳🇱

leouieda added a commit that referenced this pull request Feb 16, 2024
The bot will open PRs to update versions of the Actions we use. Use
version numbers instead of commits to make this easier.



<!--
Thank you for contributing a pull request to Fatiando! 💖

👆🏽 ABOVE: Describe the changes proposed and WHY you made them.

👇🏽 BELOW: Link to any relevant issue or pull request.

Please ensure you have taken a look at the CONTRIBUTING.md file 
in this repository (if available) and the general guidelines at 
https://github.com/fatiando/community/blob/main/CONTRIBUTING.md
-->

**Relevant issues/PRs:**
<!--
Example: "Fixes #1234" / "See also #345" / "Relevant to #111"
Use keywords (e.g., Fixes, Closes) to create the links and automatically
close issues when this PR is merged. 
See https://github.com/blog/1506-closing-issues-via-pull-requests
-->
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.

2 participants