-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
docs: update PSF scicookie report #12
Conversation
@xmnlab ready for a review |
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.
@EverVino I left some suggestions
Co-authored-by: Anavelyz Perez <42752529+Anavelyz@users.noreply.github.com>
Co-authored-by: Anavelyz Perez <42752529+Anavelyz@users.noreply.github.com>
Co-authored-by: Anavelyz Perez <42752529+Anavelyz@users.noreply.github.com>
Thanks @Anavelyz. Nice suggestions |
@xmnlab ready for a final review |
not fit into its package scope either, despite this we make sure to comply | ||
with the quality requirements (tests, CI configuration, documentation), we | ||
have also initiated communications with | ||
[Leah Wasser](https://github.com/lwasser), CEO of pyOpenSci, who has directly |
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.
it would be better to change from CEO to Executive Director .. my impression is that CEO is used more for companies
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.
just added a small comment here.
other than that, it looks good to me
Thanks @xmnlab. Fixing that and merging |
No description provided.