Skip to content

Conversation

lparsons
Copy link
Member

Description

Checklist:

  • I have previewed changes locally or with CircleCI (runs when PR is created)
  • I have completed any content reviews, such as getting input from relevant working groups. If no, please note this and wait to post the PR to the #website channel until the content has been settled.

When you are ready for a technical review/merge, post the for the link for the PR in the US-RSE Slack (#website) to ask for reviewers.

@lparsons lparsons marked this pull request as ready for review February 19, 2025 22:34
@crd477 crd477 merged commit e0d62a5 into USRSE:main Feb 20, 2025
2 checks passed
@crd477
Copy link
Contributor

crd477 commented Feb 20, 2025

Whoops, sorry for the hasty merge. If someone sees a problem, I'll fix it.

@exoticDFT
Copy link
Member

Thanks @crd477 I was just about to look at it. I'll take a second look anyway and let you know if I notice anything to adjust.

@crd477
Copy link
Contributor

crd477 commented Feb 20, 2025

I somehow totally missed Lance's comment until after I checked the previews.

@exoticDFT
Copy link
Member

Looks good to me too! Thanks again for looking at it and merging it in @crd477

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants