Skip to content
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

pattern/not-invented-here #64

Merged
merged 23 commits into from
Dec 10, 2021
Merged

pattern/not-invented-here #64

merged 23 commits into from
Dec 10, 2021

Conversation

nyeates
Copy link
Contributor

@nyeates nyeates commented Mar 16, 2017

Perfectly good solutions are being rejected because of Not Invented Here. Engineers and their managers will choose to implement functionality themselves first, even if an alternative exists.

@nyeates nyeates added 0 - Incomplete 1-initial Donuts, Early pattern ideas, ... (Please see our contribution handbook for details) labels Mar 16, 2017
From 2017-03-16 webinar
@nyeates nyeates added 2-structured Patterns with existing instances (Please see our contribution handbook for details) and removed 1-initial Donuts, Early pattern ideas, ... (Please see our contribution handbook for details) labels Mar 16, 2017
Revised, filled in gaps and cleaned up text.
Team will review & resolve further
not-invented-here.md Outdated Show resolved Hide resolved
team review, revisions & notes - Nov 2nd meeting
@maxcapraro maxcapraro added the Stale We mark issues as stale after 90 days of inactivity. This does not make any judgement about value. label Apr 21, 2020
@spier spier added the 📖 Type - Content Work Working on contents is the main focus of this issue / PR label Dec 26, 2020
@spier spier added 1-initial Donuts, Early pattern ideas, ... (Please see our contribution handbook for details) and removed 2-structured Patterns with existing instances (Please see our contribution handbook for details) labels Jan 30, 2021
@spier spier removed the Stale We mark issues as stale after 90 days of inactivity. This does not make any judgement about value. label Jan 30, 2021
@spier
Copy link
Member

spier commented Feb 6, 2021

This pattern seems to have been created at one of the early ISC Summits as well, as it contains many notes from conversations that seem to have happened in-person.

To get this pattern into a mergeable state, one would have to remove those notes and rework the content such that it becomes understandable for anybody interested in the Not-Invented-Here Mindset in the context of InnerSource.

@spier
Copy link
Member

spier commented Dec 10, 2021

I changed my mind on this PR.

The pattern is certainly in an early draft state e.g. it contains many notes that might have been written at an ISC Summit where this draft was ideated.

However I still think that this draft would create more value if we merge it into main, as that makes it more likely for readers to find it.

README.md Outdated Show resolved Hide resolved
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.

Approving this PR on the premise that:
a) the PR is so old that it is unlikely that the original authors will still work on it and
b) it is better to have a half-finished pattern draft in the main branch, than in a PR, as readers are more likely to find it in the main branch

@spier spier merged commit 712b6c4 into main Dec 10, 2021
@spier spier deleted the pattern/not-invented-here branch December 10, 2021 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-initial Donuts, Early pattern ideas, ... (Please see our contribution handbook for details) 📖 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.

None yet

7 participants