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 Draft] Internal Developer Platform - replaces PR #720 #761

Merged
merged 21 commits into from
Jan 17, 2025
Merged

Conversation

spier
Copy link
Member

@spier spier commented Jan 17, 2025

Created this PR to integrate the proposed improvements to #720.
(I wasn't able to push changes to the PR itself)

@spier spier added 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 labels Jan 17, 2025
@spier spier merged commit 19ba26e into main Jan 17, 2025
9 checks passed
@spier spier deleted the 720-replace branch January 17, 2025 21:57
@spier
Copy link
Member Author

spier commented Jan 17, 2025

Hello @caldeirav,

I used this PR here to take your amazing work from #720, apply a couple of fixes, and then integrate this into our repo. I applied GitHubs Co-authored-by property, so the contribution should be properly attributed to you :)

You now find your new pattern here:
https://github.com/InnerSourceCommons/InnerSourcePatterns/blob/main/patterns/1-initial/internal-developer-platform.md

Looking forward to keep improving this with you and the rest of the community in future PRs :)

Thank you again for the great work on this ❤️

@caldeirav
Copy link
Contributor

Thank you @spier for your guidance and taking this work to a published state.

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
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants