Create introducing-metrics-in-innersource.md#497
Create introducing-metrics-in-innersource.md#497Trueblueaddie wants to merge 10 commits intoInnerSourceCommons:mainfrom
Conversation
|
hi @Trueblueaddie. Thanks for adding this pattern. There is a pattern with the same name here: If yours is a new pattern, I would suggest to pick a different name, so that we can differentiate both patterns. I would further suggest to move this pattern to folder Please let me know if you need help with any of this, I would be happy to. |
|
I've reviewed the markdownlint and attempted to correct the spacing concerns. I'm not sure exactly what is necessary to be adjusted here. |
|
@spier thanks for calling that out. I'll see if mine has anything to add the one you shared. |
Retrieve known instance from InnerSourceCommons#497
…nstead of new patterns) (#526) * All forces mentioned were against the pattern. I'm adding some in favor for balance and better justification. * Add known instance - based on #497 * Structure the solution. Mention outliers (thank you, Addie). * Links to metrics references that are mentioned in the text. Co-authored-by: Trueblueaddie <[email protected]> Co-authored-by: Sebastian Spier <[email protected]>
|
@fioddor based on #526 I understand that we can close this PR here as duplicate, right? Just saw that this PR mentions two other authors as well. I know Daniel but Alberto is possibly from Santander Bank itself: Should we add those two to the "Acknowledgement" on the pattern? |
I guess so. WDYT, @Trueblueaddie?
Alberto is a former bitergian and currently a university professor.
Yes. |
|
Added further acknowledgements via bf274fd |
|
PR is stale, and unclear if there is anybody who wants to push it forward. |
No description provided.