-
Notifications
You must be signed in to change notification settings - Fork 65
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 pydov to package list, and the group of DOV-Vlaanderen as packa… #58
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 👍
Thank you @pjhaest !!! Just checking: I feel like we'd want to keep track of the people that were directly involved in making PyOpenSci a community, although maybe you had discussions about this and felt like it was best to credit your whole group as maintainers. I don't mean to make any more work for anybody, but just thought I should bring that up before I click merge. |
We discussed this but were a little unsure on how to include all our contributers so went for DOV-Vlaanderen instead. Looking more closely now, I see that we maybe can make the distinction between maintainers and contributers and have our three names instead. I think for us it doesn't really matter, but if you prefer our names we can shurely adapt the PR accordingly. |
I see, thank you not trying to complicate things but I think per our spec, you would all have the type "PackageMaintainer". Right now this is defined as "someone who has submitted a package that successfully went through review". Even though some people added themselves as so, if you're okay with it, I propose:
I will make the executive decision this is ok for now and if @lwasser feels strongly otherwise (I don't think she will) then I can fix with another PR later |
@NickleDave , thanks for the quick notice! Indeed, it was not entirely clear to us what would be the preferred reference, therefore we picked the entire group. @Roel , @stijnvanhoey, if it's ok with you, I can change the PR to include you as individual PackageMaintainers? |
@pjhaest all fine for me. Thanks for fixing this during the sprint! I totally missed this to-do on the pyopensci process. |
thank you all for this pr!! @stijnvanhoey @NickleDave @pjhaest i'm happy to merge or @NickleDave would you like to merge? |
oh whoops didn't see that @pjhaest made the changes merged now! excellent, you are all officially contributors!!! thank you so much. |
Thanks for the good review and the acceptance of pydov (see software review)!
We already included the badge on our GitHub and hereby provide the details of the package and the maintainers.
If we missed something, let us know and we'll make the necessary changes.