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

Added myself and Kristian Evers to AUTHORS file; Removed attribution … #558

Merged
merged 2 commits into from Aug 21, 2017
Merged

Added myself and Kristian Evers to AUTHORS file; Removed attribution … #558

merged 2 commits into from Aug 21, 2017

Conversation

busstoptaktik
Copy link
Member

…for my 1999 contribution from proj.c

Knud Poder

Kristian Evers <kreve@sdfe.dk>
Thomas Knudsen <thokn@sdfe.dk>
Copy link
Member

Choose a reason for hiding this comment

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

What is going on with that weird line-ending? Is it because of mixed CR/CR-LF?

Copy link
Member Author

Choose a reason for hiding this comment

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

Actually just a missing lf before eof. Also removed some whitespace at eol

@busstoptaktik
Copy link
Member Author

busstoptaktik commented Aug 18, 2017 via email

@busstoptaktik busstoptaktik merged commit e044eb0 into OSGeo:master Aug 21, 2017
@busstoptaktik busstoptaktik deleted the attribution branch August 21, 2017 09:02
@kbevers
Copy link
Member

kbevers commented Aug 21, 2017

Just noticed that @rouault is missing from the list!

@busstoptaktik
Copy link
Member Author

Definitely - and probably a few more: @QuLogic, @micahcochran - anyone else? But despite immodestly adding myself and @kbevers, I did not do so unitl I had stated my intention in a private email to @hobu.

And thinking it over again, I believe adding people to the AUTHORS file should probably be the sole privilege of the maintainer? @hobu: Do you agree?

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.

None yet

2 participants