-
Notifications
You must be signed in to change notification settings - Fork 98
ci: CODEOWNERS - sync to reality #3935
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
base: develop
Are you sure you want to change the base?
Conversation
Removed several code owners from inputFiles and coreComponents directories, and updated ownership for specific files.
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Nicola Castelletto <[email protected]>
Co-authored-by: Jian Huang <[email protected]>
Co-authored-by: Jian Huang <[email protected]>
Co-authored-by: Jian Huang <[email protected]>
Co-authored-by: Jian Huang <[email protected]>
Co-authored-by: Jian Huang <[email protected]>
Co-authored-by: Jian Huang <[email protected]>
Co-authored-by: Jian Huang <[email protected]>
MelReyCG
left a comment
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.
| * @rrsettgast | ||
|
|
||
| /.github/ @rrsettgast @paveltomin | ||
| /.github/ @rrsettgast |
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.
IIUC, only one person will be able to edit this file in the future
Could a 2nd person be added here? (at least, in order to have a backup solution to not let future PRs stuck)
@castelletto1 ? @herve-gross ? Someone else ?
Co-authored-by: Jian Huang <[email protected]>
| /src/coreComponents/constitutive/diffusion @paveltomin @CusiniM @dkachuma | ||
| /src/coreComponents/constitutive/dispersion @paveltomin @CusiniM @dkachuma |
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 worked on both if we need redundancy.
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.
@jafranc please add yourself where you feel comfortable, i think everyone would only appreciate that
| /src/coreComponents/constitutive/fluid @dkachuma | ||
| /src/coreComponents/constitutive/permeability @dkachuma @jhuang2601 | ||
| /src/coreComponents/constitutive/pvtPackage @dkachuma | ||
| /src/coreComponents/constitutive/relativePermeability @dkachuma |
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.
Same here :)
| /src/coreComponents/constitutive/capillaryPressure @dkachuma | ||
| /src/coreComponents/constitutive/contact @rrsettgast @jhuang2601 @jafranc | ||
| /src/coreComponents/constitutive/diffusion @dkachuma | ||
| /src/coreComponents/constitutive/dispersion @dkachuma |
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.
| /src/coreComponents/constitutive/dispersion @dkachuma | |
| /src/coreComponents/constitutive/dispersion @dkachuma @jafranc |
| /src/coreComponents/constitutive @rrsettgast @dkachuma | ||
| /src/coreComponents/constitutive/capillaryPressure @dkachuma | ||
| /src/coreComponents/constitutive/contact @rrsettgast @jhuang2601 @jafranc | ||
| /src/coreComponents/constitutive/diffusion @dkachuma |
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.
| /src/coreComponents/constitutive/diffusion @dkachuma | |
| /src/coreComponents/constitutive/diffusion @dkachuma @jafranc |
| /src/coreComponents/constitutive/fluid @dkachuma | ||
| /src/coreComponents/constitutive/permeability @dkachuma @jhuang2601 | ||
| /src/coreComponents/constitutive/pvtPackage @dkachuma | ||
| /src/coreComponents/constitutive/relativePermeability @dkachuma |
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.
| /src/coreComponents/constitutive/relativePermeability @dkachuma | |
| /src/coreComponents/constitutive/relativePermeability @dkachuma @jafranc |
No description provided.