Skip to content

Admonition on office hours page#320

Merged
jukent merged 5 commits intoNCAR:mainfrom
jukent:admonition
Jun 17, 2025
Merged

Admonition on office hours page#320
jukent merged 5 commits intoNCAR:mainfrom
jukent:admonition

Conversation

@jukent
Copy link
Collaborator

@jukent jukent commented Jun 17, 2025

Update with date and link to blog post

office-hours.md Outdated
```{admonition} ESDS Collaboratory on Wednesday, June 25!
:class: admonition
ESDS is organizing in-person "Collaboratory" days to encourage outreach, engagement, and the sharing of ideas between teams. These sessions will rotate through the ML, CG, and FL libraries (each lab housing one event per quarter). More information to come.
ESDS is organizing in-person "Collaboratory" days to encourage outreach, engagement, and the sharing of ideas between teams. These sessions will rotate through the ML, CG, and FL libraries (each lab housing one event per quarter). [For more information, chheck out this blog post.](posts/2025/collaboratory.html)
Copy link
Contributor

Choose a reason for hiding this comment

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

This line has a small typo (chheck instead of check), but otherwise looks good!

Copy link
Contributor

@mnlevy1981 mnlevy1981 Jun 17, 2025

Choose a reason for hiding this comment

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

This is more personal preference than anything else, but should the link just be the phrase "this blog post" instead of the whole sentence?

For more information, check out [this blog post](...).

Copy link
Contributor

@mnlevy1981 mnlevy1981 left a comment

Choose a reason for hiding this comment

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

LGTM!

@jukent jukent merged commit 8db9376 into NCAR:main Jun 17, 2025
2 checks passed
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.

2 participants