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: new blog post - czi funding announcement #305

Merged
merged 7 commits into from
Jan 3, 2024
Merged

Conversation

lwasser
Copy link
Member

@lwasser lwasser commented Dec 29, 2023

NOTE: This blog post needs to be renamed before merging so we can use the live preview feature setup here.


Copy link
Contributor

@kierisi kierisi left a comment

Choose a reason for hiding this comment

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

added my thoughts - mostly grammatical and flow suggestions!

_posts/2023-12-29-czi-funds-pyopensci-2024.md Outdated Show resolved Hide resolved
empowerment enables open science to reach its full potential.

### Open source maintainers need more support
Despite the importance of open source software and inclusion to fundamental
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not entirely clear what we're trying to say here. What's throwing me is the phrase "and inclusion"

Copy link
Member Author

Choose a reason for hiding this comment

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

i hear you. I think the rewrite goal was to talk about

  1. how open source is important to open science
  2. inclusion is important to both

maybe there is a way to make that more clear? there are papers about the importance of dei in science and team work in general. but thing link of ideas is a bit tricky

Copy link
Member Author

Choose a reason for hiding this comment

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

ok i implemented a minor fix and just dropped the inclusion as it really didn't make sense there!

### Open source maintainers need more support
Despite the importance of open source software and inclusion to fundamental
open science principles, open source maintainers do not get the support they
need. Maintainers need both institutional and community support in learning
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
need. Maintainers need both institutional and community support in learning
Maintainers need both institutional and community support in learning
and engaging in open source software development, as well as in maintaining tools and interacting
with the broad user base that uses these tools in support of open science.

Copy link
Member Author

Choose a reason for hiding this comment

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

can you help me understand the intended change here @kierisi so i can apply it?

Copy link
Contributor

Choose a reason for hiding this comment

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

oh! yes, for some reason it wouldn't let me select more than a single line for review. so what's showing up in green are my suggested changes, but it's not showing up as a diff

Copy link
Member Author

Choose a reason for hiding this comment

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

oh interesting! maybe we can look at that together to see why :) sometimes the github gui can be a bit quirky!

_posts/2023-12-29-czi-funds-pyopensci-2024.md Outdated Show resolved Hide resolved
_posts/2023-12-29-czi-funds-pyopensci-2024.md Outdated Show resolved Hide resolved
_posts/2023-12-29-czi-funds-pyopensci-2024.md Outdated Show resolved Hide resolved
_posts/2023-12-29-czi-funds-pyopensci-2024.md Outdated Show resolved Hide resolved
_posts/2023-12-29-czi-funds-pyopensci-2024.md Outdated Show resolved Hide resolved
_posts/2023-12-29-czi-funds-pyopensci-2024.md Outdated Show resolved Hide resolved
_posts/2023-12-29-czi-funds-pyopensci-2024.md Outdated Show resolved Hide resolved
@lwasser
Copy link
Member Author

lwasser commented Dec 29, 2023

ok!! i think this should be ready to go :) i'll rename locally and we can still accept minor edits with an intended publish date of next week when we can back from the holiday!

lwasser and others added 4 commits December 29, 2023 15:12
Fix: install gems locally

Fix: install gems locally

Fix: install gems locally

Fix: another try
Co-authored-by: Jesse Mostipak <[email protected]>

Update _posts/2023-12-29-czi-funds-pyopensci-2024.md

Co-authored-by: Jesse Mostipak <[email protected]>
Co-authored-by: Jesse Mostipak <[email protected]>

Apply suggestions from code review

Co-authored-by: Jesse Mostipak <[email protected]>

Fix: minor edits after review

Fix: Rename for jan 3 publication

Fix: Rename for jan 3 publication
@lwasser lwasser merged commit 52f69f5 into pyOpenSci:main Jan 3, 2024
1 check failed
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