GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||||||||||||||||||||||||||
GeezerVison is showing github.com — Create assisted_compliance.md by NewMexicoKid · Pull Request #74 · InnerSourceCommons/InnerSourcePatterns Conversation
Collaborator
This is: Helping repo owners be compliant by writing their CONTRIBUTING.md for them as a pull request.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
This is: Helping repo owners be compliant by writing their CONTRIBUTING.md for them as a pull request
rrrutledge
reviewed
Uh oh!
There was an error while loading. Please reload this page.
gruetter
requested changes
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Quite a bit of work to be done ...
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsUh oh!
There was an error while loading. Please reload this page. Uh oh!
There was an error while loading. Please reload this page.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The way the context is written make me think that the pattern only applies in a very narrow case. I feel we need to abstract a few things to make it more applicable.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you suggest some ways to abstract it? Is the key restrictive context the mandatory nature of the CONTRIBUTING.md file?
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Looks like context to me
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsUh oh!
There was an error while loading. Please reload this page.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I propose to make this less conditional.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsThere was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @gruetter, I agree: IMHO, beneficiaries of an automated PR with a CONTRIBUTING.md doc should be viewed as an act of service, not enforcement. Maybe pitch this as a benefit, e.g.,
As you can see on the InnerSourcePattern repository's "Community profile" page, GitHub considers CONTRIBUTING one of several recommended community standards:
Once people realize value of these documents--e.g., "A good README.md file will help people discover your projects from Intranet search," teams start to realize that CONTRIBUTING docs are a benefit instead of a chore. On the other hand, some people just don't want to participate for one reason or another.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. While it is ideal to persuade rather than to enforce, the fact that this pattern was written shows that persuasion may not be sufficient. I agree that getting people to understand the benefits should be something tried first (and this should be part of the pattern).
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsThere was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks for the clarification, @NewMexicoKid.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @gregswindle I know that this conversation here is a loooong while back. I just wanted to let you know what we have merged the pattern to the main line by now. I am also curious to learn more about your project https://github.com/commonality/generator-community and what experiences you made with it. If you would like to chat about it, feel free to reply here or join the community in https://innersourcecommons-inviter.herokuapp.com/ :)
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsUh oh!
There was an error while loading. Please reload this page.
Member
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
9 tasks done
Member
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
rrrutledge
reviewed
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Added commit suggestion, @lenucksi.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsUh oh!
There was an error while loading. Please reload this page. Co-authored-by: rrrutledge <[email protected]>
lenucksi
requested review from
cewilliams,
lenucksi and
nyeates
as code owners
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Member
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
spier
reviewed
Uh oh!
There was an error while loading. Please reload this page.
spier
reviewed
Uh oh!
There was an error while loading. Please reload this page.
spier
reviewed
Uh oh!
There was an error while loading. Please reload this page.
spier
reviewed
Uh oh!
There was an error while loading. Please reload this page. …moving the entry to the right group.
Member
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
Contributor
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
|
There was an error while loading. Please reload this page.
|
Reaching timer. Merging. I added a reference section, pointing to some interesting discussions in this PR. One more draft pattern from the old days (2017) merged into the mainline. 🎉 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.