Skip to content
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

Add CODEOWNERS file #398

Merged
merged 11 commits into from
Oct 1, 2021
Merged

Add CODEOWNERS file #398

merged 11 commits into from
Oct 1, 2021

Conversation

fossell
Copy link
Contributor

@fossell fossell commented Sep 30, 2021

This is just a starting point with @WenMeng-NOAA listed as a global owner who will be assigned as reviewer to all PRs unless the PR only makes mods to docs/ or community run_upp scripts, in which case DTC members will be added to review PR. This file can be modified as needed to best suit code managers needs.

This PR closes #397

@fossell fossell added the enhancement New feature or request label Sep 30, 2021
@WenMeng-NOAA WenMeng-NOAA added Ready for Review This PR is ready for code review. No Baseline Change No baseline of the UPP regression tests are made. labels Oct 1, 2021
@WenMeng-NOAA
Copy link
Collaborator

@fossell Thanks for bringing new function in UPP repos.

@WenMeng-NOAA
Copy link
Collaborator

@fossell Would you rename your fork to "UPP" though it will not impact on your PR?

@fossell
Copy link
Contributor Author

fossell commented Oct 1, 2021

@WenMeng-NOAA - Yes, fork is changed, thanks!

@WenMeng-NOAA WenMeng-NOAA merged commit 74135d2 into NOAA-EMC:develop Oct 1, 2021
EricJames-NOAA pushed a commit to EricJames-NOAA/UPP that referenced this pull request Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request No Baseline Change No baseline of the UPP regression tests are made. Ready for Review This PR is ready for code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add CODEOWNERS file
2 participants