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.

Sign up for GitHub

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

Jump to bottom

Remove Laurie from CODEOWNERS #800

Merged
merged 1 commit into from
Dec 10, 2021
Merged

Remove Laurie from CODEOWNERS #800

merged 1 commit into from
Dec 10, 2021

Conversation

llpcarson
Copy link
Contributor

Remove Laurie from CODEOWNERS

grantfirl
grantfirl approved these changes Dec 8, 2021
View reviewed changes
Copy link
Collaborator

@grantfirl grantfirl left a comment

Choose a reason for hiding this comment

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

Perhaps this and adding @SamuelTrahanNOAA can be tacked on to another PR.

@climbfuji
Copy link
Collaborator

Perhaps this and adding @SamuelTrahanNOAA can be tacked on to another PR.

I can pull this PR, as much as I dislike it, into the next PR for the UFS (to be committed next).

@SamuelTrahanNOAA SamuelTrahanNOAA self-requested a review December 8, 2021 20:11
SamuelTrahanNOAA
SamuelTrahanNOAA approved these changes Dec 8, 2021
View reviewed changes
Copy link
Collaborator

@SamuelTrahanNOAA SamuelTrahanNOAA left a comment

Choose a reason for hiding this comment

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

I'm approving these changes so that Laurie can have a stress-free retirement. I'm happy to take over her PR review duties.

@climbfuji climbfuji mentioned this pull request Dec 9, 2021
Merged
climbfuji
climbfuji approved these changes Dec 9, 2021
View reviewed changes
Copy link
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

This PR has been pulled into #795 and will be merged and closed as part of it.

@climbfuji climbfuji merged commit d76a4e9 into NCAR:main Dec 10, 2021
climbfuji added a commit that referenced this pull request Dec 10, 2021
@climbfuji
Merge pull request #795 from climbfuji/greg_new_thompson_cloud_fracti…
fb752d4 
…on_with_ruiyu_cloud_cover_change_xu_randall

Wrapper PR for #781 (New Thompson cloud fraction) and #800 (Remove Laurie from CODEOWNERS)
@grantfirl grantfirl mentioned this pull request Dec 16, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SamuelTrahanNOAA SamuelTrahanNOAA SamuelTrahanNOAA approved these changes

@grantfirl grantfirl grantfirl approved these changes

@ligiabernardet ligiabernardet ligiabernardet approved these changes

@climbfuji climbfuji climbfuji approved these changes

@mkavulich mkavulich Awaiting requested review from mkavulich

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@llpcarson @climbfuji @ligiabernardet @grantfirl @SamuelTrahanNOAA