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

Stress the purpose of the OSPO #494

Merged
merged 1 commit into from
Jun 17, 2024

Conversation

cornelius
Copy link
Member

This pull request suggests to change the "what is this book about" section to stress what the purpose of an OSPO is. This is meant to make it clearer that the OSPO is a way to address the need of an organization to manage open source, not an end in itself.

There might be some more places in the book, where this could be made clearer with similar changes, but this section seems to be a good place to start with. If we agree on doing it this way, we can review and adapt other parts of the book as well, if needed.

@cornelius cornelius requested a review from a team as a code owner June 12, 2024 07:17
Copy link

netlify bot commented Jun 12, 2024

Deploy Preview for ospomindmap canceled.

Name Link
🔨 Latest commit af37158
🔍 Latest deploy log https://app.netlify.com/sites/ospomindmap/deploys/6669beec1a4e9a0008a59736

Copy link

netlify bot commented Jun 12, 2024

Deploy Preview for ospobook ready!

Name Link
🔨 Latest commit af37158
🔍 Latest deploy log https://app.netlify.com/sites/ospobook/deploys/6669beecfbe9500008afdaea
😎 Deploy Preview https://deploy-preview-494--ospobook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

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

I like this change a lot! Thanks @cornelius.

Note: Seems like DCO robot is complaining

Signed-off-by: Cornelius Schumacher <[email protected]>
@cornelius cornelius force-pushed the what-is-the-book-about branch from b1a2d76 to af37158 Compare June 12, 2024 15:29
@cornelius
Copy link
Member Author

Thanks. I fixed the DCO.

Copy link
Member

@anajsana anajsana left a comment

Choose a reason for hiding this comment

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

LGTM!

@anajsana anajsana merged commit c8b31c8 into todogroup:main Jun 17, 2024
6 checks passed
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.

4 participants