-
Notifications
You must be signed in to change notification settings - Fork 196
Concept Anchor - Formal pattern improvements #530
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
Conversation
fioddor
commented
Mar 9, 2023
- Move pattern status to its section
- Fill in the resulting context
- Move pattern status to its section - Fill in the resulting context
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love the incremental approach you are taking to improving the patterns!
Left some comments inline.
Several wording and style fixes. Co-authored-by: Sebastian Spier <github@spier.hu>
Thanks for your review, @spier. I'm discovering that I'm multitasking too much. You are catching embarrasing obvious flaws that slip through my context juggling. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
OpenChain Project has created short Explainers (https://github.com/OpenChain-Project/Reference-Material/tree/master/Education-For-Internal-Teams) and it may be worth having a look at those and get some ideas for this InnerSource pattern. OpenChain Project explainers are for
|
@winterrocks are you proposing to write similar explainers for InnerSource, and add them to the Concept Anchor pattern? Btw how did you come across this old PR? :) |
That may be one idea. I just threw the idea for a wider audience here. How I came across this PR, I do not remember now - pure luck... |