Skip to content

Conversation

@dhomeier
Copy link
Contributor

@dhomeier dhomeier commented Feb 15, 2024

Part of the transition of our Affiliated Package Review process into partnership with pyOpenSci

Fixing #569

Copy link
Member

@pllim pllim left a comment

Choose a reason for hiding this comment

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

I think the content is good, except for a typo.

Would be nice if @hamogu and @lwasser could have a look through. Thanks!

Co-authored-by: P. L. Lim <2090236+pllim@users.noreply.github.com>
Copy link
Member

@pllim pllim left a comment

Choose a reason for hiding this comment

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

This is much better than the version I came up with over at #573 .

When should we merge this?

@dhomeier dhomeier marked this pull request as ready for review February 17, 2024 00:03
Copy link
Member

@pllim pllim left a comment

Choose a reason for hiding this comment

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

We already started the transition here and there, and it has been 2 weeks with multiple approvals and no objection, so I am merging. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants