Skip to content

Conversation

zkoppert
Copy link
Member

@zkoppert zkoppert commented Mar 8, 2022

doing this based off of the discussion here

Copy link
Member

@spier spier left a comment

Choose a reason for hiding this comment

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

Thanks for starting this @zkoppert.

Can you explain why the Engagement Model is key? That part is the only thing that is different from the purpose+principles structure of the other 2 orgs, that's why it caught my eye.

@zkoppert
Copy link
Member Author

zkoppert commented Mar 8, 2022

Thanks for starting this @zkoppert.

Can you explain why the Engagement Model is key? That part is the only thing that is different from the purpose+principles structure of the other 2 orgs, that's why it caught my eye.

I've attempted to do that here. Let me know if it needs more information.

@zkoppert
Copy link
Member Author

Hey @spier! Any changes you would like to see here?

@spier
Copy link
Member

spier commented Mar 20, 2022

My bad. Was looking at this in the GitHub mobile app some time ago and couldn't see the whole file. Then I got distracted.

I do recall that I was wondering if it would be easier to follow if we put a generic description of the approach in the Solution block and then put the 3 examples of the company-specific implementations under the Known Instances.

Will try to make time for a more detailed review in the evening. But if I don't make it today then I unfortunately won't get to it before next Friday.

@zkoppert
Copy link
Member Author

Thanks @spier. Yes I added the implementation details under the known instances header. Let me know what you think, when available, on what we should add for a general description that you mentioned.

Copy link
Member

@spier spier left a comment

Choose a reason for hiding this comment

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

Left some suggestions including some text for the general description. Hope you like it :)

Further I am unsure whether to write the GitHub KI block from the "we" or "they" perspective. Bosch used the latter.

Need to review some other patterns to see if we have a similar example that we can follow.

Copy link
Member

@spier spier left a comment

Choose a reason for hiding this comment

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

Lgtm

Added two minor fixes that I will commit myself.

@spier spier merged commit f4f3796 into InnerSourceCommons:main Apr 1, 2022
@spier
Copy link
Member

spier commented Apr 1, 2022

Thank you @zkoppert for sharing the GH principles with us! This will go live in the online book momentarily.

@zkoppert zkoppert deleted the patch-1 branch April 1, 2022 21:50
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.

3 participants