-
Notifications
You must be signed in to change notification settings - Fork 201
feat: add Migrating from InnerSource to Open Source #851
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
feat: add Migrating from InnerSource to Open Source #851
Conversation
chore: add inputs from ISPO WG to Require InnerSource before Open Source
I used the opportunity to write a GitHub workflow that checks if all patterns are listed in README.md @jeffabailey the new workflow is called Overview Compete so far. Do you have a more helpful (but short) idea for naming this workflow? |
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.
As these are just initial patterns, we can merge them basically as is. We can harden them further once we find (multiple) orgs that have adopted this.
Left some inline comments below.
patterns/1-initial/migrating-from-innersource-to-open-source.md
Outdated
Show resolved
Hide resolved
patterns/1-initial/migrating-from-innersource-to-open-source.md
Outdated
Show resolved
Hide resolved
Overview Complete works, but "Ensure All Patterns Are Listed in README" may be a more direct name, or "Validate Pattern Index in README". |
refactor: adjust patlet to 2 sentence problem solution feat: add references to open source project guidance refactor: adjust problem to be more specific
…:jeffabailey/InnerSourcePatterns into migrating-from-innersource-to-opensource
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.
Pull Request Overview
This PR adds a new InnerSource pattern documenting the migration process from InnerSource to Open Source, enhancing the existing "Require InnerSource before Open Source" pattern with additional context and considerations from the ISPO Working Group.
- Adds a comprehensive new pattern "Migrating from InnerSource to Open Source" with detailed migration steps and considerations
- Updates the existing "innersource-before-open-source.md" pattern with regulatory, legal, and bidirectional movement considerations
- Includes a GitHub workflow to ensure all patterns are properly documented in the README
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
File | Description |
---|---|
patterns/1-initial/migrating-from-innersource-to-open-source.md | New pattern providing structured guidance for transitioning InnerSource projects to open source |
patterns/1-initial/innersource-before-open-source.md | Enhanced with regulatory compliance, IP risk, and OSPO review considerations |
README.md | Added reference to the new migration pattern |
.github/workflows/overview-complete.yml | New workflow to validate all patterns are documented in README |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
patterns/1-initial/migrating-from-innersource-to-open-source.md
Outdated
Show resolved
Hide resolved
I was trying to find a shorter name, like the other ones. Probably it is hard to get it to be both short and descriptive at the same time :) |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
patterns/1-initial/migrating-from-innersource-to-open-source.md
Outdated
Show resolved
Hide resolved
patterns/1-initial/migrating-from-innersource-to-open-source.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jeffabailey this is now live in the repo. Thank you for pushing the work of the ISPO WG into this repo and with that making the content available for the broader community ❤️🥳 Btw the Copilot suggestions on this PR were not super helpful in this particular case, and I ended up rejecting most of them. |
I noticed the same. It was way off. |
chore: add inputs from ISPO WG to Require InnerSource before Open Source
This PR is brought to you by the ISPO WG: https://docs.google.com/document/d/1Jh7Ig5cjXjyH0URdeZjKTtd89tz9wIxgoFZwp_vCKSY/edit?tab=t.0