Skip to content

Codeowners functional test#43

Merged
FreddyMSchubert merged 2 commits intodevfrom
codeowners-functional-test
Aug 26, 2025
Merged

Codeowners functional test#43
FreddyMSchubert merged 2 commits intodevfrom
codeowners-functional-test

Conversation

@FreddyMSchubert
Copy link
Copy Markdown
Member

Screenshot 2025-08-26 at 09 42 52

Alright seems to work very cool. Now we can give specific GitHub usernames responsibility over certain parts of the code and they will be marked as reviewers if that code was changed and can't be removed.

Config is in .github/CODEOWNERS

Dear Paul please review and merge this highly important PR

FreddyMSchubert and others added 2 commits August 26, 2025 09:35
Update CODEOWNERS to properly fix path rules
@FreddyMSchubert FreddyMSchubert self-assigned this Aug 26, 2025
@FreddyMSchubert FreddyMSchubert added the monorepo Related to the functionality of the monorepo itself, including the CI/CD pipelines label Aug 26, 2025
Copy link
Copy Markdown
Member

@PaulicStudios PaulicStudios left a comment

Choose a reason for hiding this comment

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

hell no

@PaulicStudios PaulicStudios self-requested a review August 26, 2025 07:46
@PaulicStudios
Copy link
Copy Markdown
Member

What merge rules are these that even after a review nobody can merge?

@FreddyMSchubert
Copy link
Copy Markdown
Member Author

really weird i dont know why lemme check

@FreddyMSchubert
Copy link
Copy Markdown
Member Author

Yeah that was badly configured my bad. Should be better now.

@FreddyMSchubert FreddyMSchubert merged commit d5512a0 into dev Aug 26, 2025
4 checks passed
@FreddyMSchubert FreddyMSchubert deleted the codeowners-functional-test branch August 26, 2025 07:51
@FreddyMSchubert
Copy link
Copy Markdown
Member Author

Literally nobody that didn't have bypass permissions could update. Maybe we should keep that for main though? Literally only give organization owners permission to ship stuff to prod. Sounds good Imma do that.

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

Labels

monorepo Related to the functionality of the monorepo itself, including the CI/CD pipelines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants