FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add Thales Group as Known Instance across five patterns by rrrutledge · Pull Request #909 · InnerSourceCommons/InnerSourcePatterns · GitHub

Add Thales Group as Known Instance across five patterns - #909

Merged
spier merged 12 commits into
InnerSourceCommons:mainfrom
rrrutledge:add-thales-known-instances
Jun 17, 2026
Merged

Add Thales Group as Known Instance across five patterns#909
spier merged 12 commits into
InnerSourceCommons:mainfrom
rrrutledge:add-thales-known-instances

Conversation

Copy link
Copy Markdown
Contributor

Summary

Adds Thales Group as a Known Instance across five patterns, based on Marius Moulis's InnerSource Commons talk "Why People Matter Most: Building a Sustainable InnerSource Strategy" (May 2026). Thales has run an InnerSource Program Office for 10 years across 85,000+ employees in 65 countries.

  • Praise Participants — Thales applies this pattern explicitly by name, with per-contribution recognition plus formal awards at the annual "Tales I Days" in-person event
  • Developer Incentive Alignment for InnerSource Contribution — InnerSource contributions are on the expert career path as proof of leadership and expertise; replaces "TBD" Known Instances
  • InnerSource Ambassadors — Thales "Community Leaders" meet quarterly with the ISPO as a "Steering Board," creating a structured bottom-up/top-down governance loop
  • InnerSource Hackathon — Tales I Days combines hackathon + conference + awards ceremony in one annual in-person event open to non-engineering staff as well
  • InnerSource License — Thales created four ready-to-use InnerSource licenses with legal partners, available at no cost to all participants

Test plan

  • Verify all five YouTube links resolve to the correct talk
  • Confirm each entry matches the surrounding Known Instances formatting style (bullet style, bold org name, trailing citation)
  • Check that the Thales section in InnerSource License appears correctly between GovTech and the Status section

🤖 Generated with Claude Code

Based on Marius Moulis's InnerSource Commons talk "Why People Matter Most:
Building a Sustainable InnerSource Strategy" (May 2026), Thales Group's
10-year ISPO experience is added as a Known Instance to:

- Praise Participants: Thales applies the pattern explicitly, with formal
  awards at the annual "Tales I Days" event
- Developer Incentive Alignment: InnerSource on the expert career path as
  proof of leadership/expertise (replaces "TBD" Known Instances)
- InnerSource Ambassadors: Thales "Community Leaders" / quarterly Steering
  Board as a structured bottom-up/top-down governance loop
- InnerSource Hackathon: Tales I Days combines hackathon + conference +
  awards in one annual in-person event
- InnerSource License: Thales's framework of four ready-to-use licenses
  developed with legal partners

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
rrrutledge and others added 3 commits May 27, 2026 14:11
- Fix "Tales I Days" typo -> "Thales I/O Days" in hackathon and praise-participants
- Clarify hackathon entry: full remote 2-day hackathon + in-person conference, awards open to all job families
- Remove "at no cost" from innersource-license Thales entry

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
rrrutledge and others added 3 commits June 15, 2026 10:34
Updates Thales known instances per Marius Moulis's corrections:
- developer-incentive-alignment: use his exact wording for the career path rationale
- innersource-ambassador: rename "Community Leaders" to "Steering Board members", "business unit" to "organizational unit"
- innersource-hackathon: "reward ceremony for key contributors" (not "award ceremony"), shorter description
- innersource-license: trim to his shorter approved text
- praise-participants: remove the career-path cross-reference, shorten to his approved text

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

spier commented Jun 17, 2026

Copy link
Copy Markdown
Member

For the translation leads that got notified on this PR: I only made a minor fix to one link/URL. So nothing to do for you here. Sorry for the alert :)

spier commented Jun 17, 2026

Copy link
Copy Markdown
Member

@rrrutledge I cannot tell where Marius's Jun 11 review feedback happened, but I am assuming in some other channel (slack, email, etc).

Therefore I will go ahead and merge.

Copy link
Copy Markdown
Contributor Author

Thank you! I had been iterating with Marius in Slack, and he signed off just now :)

spier commented Jun 17, 2026

Copy link
Copy Markdown
Member

Something changed in the vale style checker, which is why the GitHub Action vale.yml is failing.

I tried various things, including updating the GitHub action but so far no luck.

One lead that might cause the issue, is the folder to which our isc-styles package gets synced.

As GHA

When vale runs as a GHA as part of this, it loads the package into this folder:
SUCCESS Synced 1 package(s) to '.../InnerSourcePatterns/.github/vale'

Locally

When running this locally I see this instead
SUCCESS Synced 1 package(s) to '.../InnerSourcePatterns/.github/vale/.vale-config/styles'

As I am not able to fix this quickly, will still merge this PR here and fix the issue separately i.e. on a new branch/PR.

spier merged commit 3ebd603 into InnerSourceCommons:main Jun 17, 2026
9 of 11 checks passed
spier added 2-structured Patterns with existing instances (Please see our contribution handbook for details) 📖 Type - Content Work Working on contents is the main focus of this issue / PR labels Jun 17, 2026
spier added the 🐅 patterns-in-the-wild InnerSource patterns that were spotted in the wild. We can extract Known Instances and new patterns. label Jun 17, 2026

spier commented Jun 17, 2026
edited
Loading

Copy link
Copy Markdown
Member

@rrrutledge your changes are now live e.g. see https://patterns.innersourcecommons.org/p/innersource-hackathon#known-instances

Thank you for integrating this content from the community calls into our patterns!

rrrutledge added a commit to rrrutledge/InnerSourcePatterns that referenced this pull request Aug 11, 2026
Two corrections per review:

- Step 1 now accepts a transcript handed to it directly (the
  going-forward case: an automated source pulls a Zoom transcript once a
  community call finishes, with no YouTube URL involved at all) as well
  as a YouTube URL to fetch from - the URL-fetch path stays for backlog
  talks and one-off requests.
- Step 8's "one concern per PR" is replaced with "one talk = one PR":
  bundle everything a single talk's analysis found - Known Instances
  across multiple patterns, clarifications, new pattern drafts - into
  one PR, matching how InnerSourceCommons#909 actually added Thales across five patterns
  in a single PR. Only split PRs across genuinely different source talks.

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
spier added a commit that referenced this pull request Aug 18, 2026
* Add analyze-innersource-video Claude Code skill

Adds a Claude Code skill that analyzes an InnerSource Commons community
call video against this repo's pattern library: it fetches the talk's
transcript and metadata, surveys adjacent patterns, and categorizes the
talk's content into Known Instance candidates, clarifications to an
existing pattern, or genuinely new pattern candidates - applying a
"uniquely InnerSource" filter to screen out generic engineering advice
that isn't specific to cross-team, cross-org contribution dynamics.

The skill always proposes and asks for confirmation before editing any
pattern file or opening a PR - it never auto-commits. This was already
used manually to produce the Thales Group Known Instance PR (#909); this
commit is the first time the skill itself is checked into the repo
rather than living only on one contributor's machine.

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

* Accept a transcript directly, and bundle one talk into one PR

Two corrections per review:

- Step 1 now accepts a transcript handed to it directly (the
  going-forward case: an automated source pulls a Zoom transcript once a
  community call finishes, with no YouTube URL involved at all) as well
  as a YouTube URL to fetch from - the URL-fetch path stays for backlog
  talks and one-off requests.
- Step 8's "one concern per PR" is replaced with "one talk = one PR":
  bundle everything a single talk's analysis found - Known Instances
  across multiple patterns, clarifications, new pattern drafts - into
  one PR, matching how #909 actually added Thales across five patterns
  in a single PR. Only split PRs across genuinely different source talks.

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

* Write the whole skill as transcript-first, not YouTube-first

Per review: stop framing this as two parallel input paths ("YouTube URL
or transcript"). The skill takes a transcript - full stop; where it came
from is the caller's concern, not something this document needs to
branch on. YouTube-specific fetch mechanics (youtube-transcript-api,
yt-dlp) stay as a practical note for when a transcript still needs to be
pulled from a video, not as the document's primary framing. Generalizes
metadata field descriptions and "cite the source link" away from
video-specific phrasing, and retitles the output template's "Video
summary" to "Talk summary" to match.

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

* Stop discouraging new-pattern drafts from a single talk

The old guidance ("one talk is one data point, not proof a pattern is
widely applicable") read as a reason to hold back drafting a genuine
new-pattern candidate. But this process only ever sees one talk at a
time, so that standard would mean a new pattern candidate never gets
drafted at all. Checked meta/contributor-handbook.md: maturity level 1
(Initial) has NO validation requirement - it's explicitly for a single
unstructured idea. One instance is what's needed for level 2
(Structured); only 3+ need level 3 (Validated). So the guidance now says
to draft at Initial from one talk, which is exactly what that level is
for - confirmed against Paired Onboarding Sprint (#926), the first
pattern this skill has drafted this way.

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

* Read README's pattern list instead of extracting patlets

Per Sebastian Spier's review: the repo's own README.md already has a
"List of Patterns" section with every pattern's title and patlet, kept
current as patterns are added (spot-checked against the newest merged
pattern - it's there). Reading that one file directly is simpler and
more reliable than writing and running a script to walk the patterns/
directory and regex out each Patlet section.

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: Sebastian Spier <github@spier.hu>

* Require checking a pattern's actual Solution before citing a Known Instance

Topical keyword overlap (e.g. 'developer environment') isn't enough - the
GDK/Internal Developer Platform mismatch on #926 happened because a local
dev tool got matched to a pattern about centralized, deployed platforms
purely on theme.

---------

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: Sebastian Spier <github@spier.hu>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2-structured Patterns with existing instances (Please see our contribution handbook for details) 📖 Type - Content Work Working on contents is the main focus of this issue / PR 🐅 patterns-in-the-wild InnerSource patterns that were spotted in the wild. We can extract Known Instances and new patterns.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL