-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update code owners to teams #87
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #87 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 5 5
Lines 127 127
Branches 16 16
=========================================
Hits 127 127 Continue to review full report at Codecov.
|
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.
Doesn't seem to work like this (see bbtsoftware/BBT.StrategyPattern#36)
I don't get it, bbtsoftware/BBT.StrategyPattern#36 is about using |
bbtsoftware/BBT.StrategyPattern#36 was mainly my test if automatic review request are working, which failed. But in the meantime I set teams to public, which might have been the issue |
Ah I see. |
Nope, still not working (bbtsoftware/BBT.StrategyPattern#38) |
Alright, abandoning this in favour of a working solution then? |
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.
I made the required changes to the team settings so that this should work now
Update code owners to teams for easier maintenance