Skip to content

Adds a pattern draft for making governance levels explicit. #281

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

Merged
merged 14 commits into from
Feb 3, 2021

Conversation

MaineC
Copy link
Member

@MaineC MaineC commented Jan 20, 2021

This adds a pattern for making several levels of openess wrt. governance levels
explicit to project contributors. Inspired by several versions of Open Source
project governance (e.g. single vendor with full control, foundation hosted with
vendor neutrality as a goal etc.) this pattern tries to translate these versions
into an InnerSource context.

This adds a pattern for making several levels of openess wrt. governance levels
explicit to project contributors. Inspired by several versions of Open Source
project governance (e.g. single vendor with full control, foundation hosted with
vendor neutrality as a goal etc.) this pattern tries to translate these versions
into an InnerSource context.
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.

Great first version @MaineC!
I would be happy to merge that in as a level 1 (Initial) pattern.

I sent some inline suggestions for you to review.

Update:
I just noticed that a Patlet is missing. Maybe you confused Patlet and Context?

welcome to read the code. They can submit feature requests and bug reports for
things they would like to see changed.

* Contributions welcome: People outside the core development team may use the
Copy link
Member

Choose a reason for hiding this comment

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

Is "core development team" here the same as "backing team" which you used above?

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 think I'm lacking a good term here: What I'm trying to explain is that there is a difference between a project that is carried forward by individuals working under one line manager, delivering one roadmap vs. a project where individuals from multiple such teams collaborate.

Copy link
Member

Choose a reason for hiding this comment

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

I see. I don't know the best term here either.
For the time being I would then suggest to just introduce and define a term, likely towards the beginning of the Pattern, and then stick to that term through out the pattern.

As we improve this pattern over time we might come up with another term, possibly influenced by other patterns as well. We may even want to pull such terms that we attribute certain meaning to into a central glossary to help pattern authors and readers to use these terms in a similar manner.

@spier
Copy link
Member

spier commented Jan 20, 2021

Also I committed and resolved some of my own suggestions that were only required to make our Markdownlint check happy :)

@spier
Copy link
Member

spier commented Jan 21, 2021

@MaineC I took the liberty to resolve/merge some of my suggested edits that I did not consider controversial. Hope I didn't overstep there, so feel free the change them back if you feel strongly about any of them.

@MaineC
Copy link
Member Author

MaineC commented Jan 22, 2021

Oh - that's what caused those pesky merge conflicts when I tried to push the changes I made according to your review yesterday :/

@MaineC
Copy link
Member Author

MaineC commented Jan 22, 2021

Are there any more open suggestions from your side that I should address?

@MaineC
Copy link
Member Author

MaineC commented Jan 22, 2021

One more caveat: Please leave this pull request open for a few more days, I've asked a few people via Twitter whether they'd have time to add their perspective.

Edit: One piece of background to integrate here: https://twitter.com/galoppini/status/1352270671011835905

Edit 2: One very current case where we could draw inspiration from here: https://twitter.com/MaineC/status/1352123851237429248

Edit 3: https://twitter.com/MaineC/status/1351913663020617732 ... this is the origin of this pull request for comparison

Any help with integrating the different perspectives welcome. :)

@spier
Copy link
Member

spier commented Jan 22, 2021

@MaineC only two more suggestions from my end. See:
https://github.com/InnerSourceCommons/InnerSourcePatterns/pull/281/files

And great job bringing in more people into the conversation!

Co-authored-by: Sebastian Spier <github@spier.hu>
@MaineC
Copy link
Member Author

MaineC commented Jan 25, 2021

Side note: After going through the paper linked above I believe we can come up with a few more fine-grained patterns along the lines of "If you want to increase contributor engagement, try making your InnerSource project more transparent and more accessible".

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!

@spier
Copy link
Member

spier commented Jan 25, 2021

One more caveat: Please leave this pull request open for a few more days, I've asked a few people via Twitter whether they'd have time to add their perspective.

Edit: One piece of background to integrate here: https://twitter.com/galoppini/status/1352270671011835905

Edit 2: One very current case where we could draw inspiration from here: https://twitter.com/MaineC/status/1352123851237429248

Edit 3: https://twitter.com/MaineC/status/1351913663020617732 ... this is the origin of this pull request for comparison

Any help with integrating the different perspectives welcome. :)

@MaineC this PR looks great to me now. Do you still want to keep it open, or shall I merge it?

@spier spier self-assigned this Jan 25, 2021
@spier spier added the 📖 Type - Content Work Working on contents is the main focus of this issue / PR label Jan 29, 2021
@spier
Copy link
Member

spier commented Jan 29, 2021

@MaineC shall we merge this PR, even if only to get the pattern "out there" in it's initial form and make it easier to link to it?

Also do you want to add yourself to the pattern in a ## Author section?

@MaineC
Copy link
Member Author

MaineC commented Feb 3, 2021

Feel free to merge the pattern. I think we can add additional detail as separate patterns.

@spier
Copy link
Member

spier commented Feb 3, 2021

@MaineC excellent. Merging now.

@spier spier merged commit 2402f05 into InnerSourceCommons:master Feb 3, 2021
@MaineC MaineC deleted the governance-levels branch November 8, 2021 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📖 Type - Content Work Working on contents is the main focus of this issue / PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants