Skip to content

Move and fix CODEOWNERS#73

Merged
eFFeeMMe merged 3 commits into
masterfrom
fix-codeowners
Jun 19, 2026
Merged

Move and fix CODEOWNERS#73
eFFeeMMe merged 3 commits into
masterfrom
fix-codeowners

Conversation

@eFFeeMMe

Copy link
Copy Markdown
Contributor

claims-resolution is no more, claims-settlement is the one I was thinking of.

Also I moved the CODEOWNERS file to the root folder while I was at it.

@eFFeeMMe eFFeeMMe requested review from a team as code owners June 19, 2026 12:25
@eFFeeMMe eFFeeMMe requested review from claudio-dalicandro and nicflower and removed request for a team June 19, 2026 12:25
@nicflower

Copy link
Copy Markdown

are we sure this is on us? @neslinesli93 @korsmakolnikov 🤔

@eFFeeMMe

Copy link
Copy Markdown
Contributor Author

are we sure this is on us? @neslinesli93 @korsmakolnikov 🤔

Good question... When fixing the original configuration which had dead teams in it etc. I ended up giving ownership to everybody in Italy to play it safe, but you can remove yourselves if you're not really involved with this.

manueledones
manueledones previously approved these changes Jun 19, 2026
Comment thread CODEOWNERS Outdated
Comment thread CODEOWNERS Outdated
@@ -0,0 +1,5 @@
* @primait/claims-operations @primait/claims-settlement @primait/claims-gateway

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we add sres also here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think they care about the Elixir stuff, but you did remind me to add them to /.github! Lemme know if this addresses your comment

@chess4ever

Copy link
Copy Markdown
Contributor

We use CSV schema also in the CGP. I would probably ad UK and Spain as well

@eFFeeMMe eFFeeMMe changed the title Move and fix CODEOWNERS (I had the wrong team name) Move and fix CODEOWNERS Jun 19, 2026
@eFFeeMMe eFFeeMMe merged commit 019f3f6 into master Jun 19, 2026
6 checks passed
@eFFeeMMe eFFeeMMe deleted the fix-codeowners branch June 19, 2026 14:19
@eFFeeMMe

Copy link
Copy Markdown
Contributor Author

I'm optimistically merging even though there's an extant comment from FraZ, I'll open another PR if he requests further changes!

Also I'm asking company-iam to remove the dead teams from Personio so that I can avoid mentioning dead teams in CODEOWNERS again 😂

@eFFeeMMe

Copy link
Copy Markdown
Contributor Author

Turns out that company-iam is not in charge of teams at the Personio level (which float downstream to GitHub and Okta). I've asked the People team.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants