Skip to content

Update schedule, add roles and team - #16

Merged
ateucher merged 21 commits into
mainfrom
schedule-roles
Oct 14, 2026
Merged

ateucher merged 21 commits into
mainfrom
schedule-roles

Conversation

@ateucher

@ateucher ateucher commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

This:

@github-actions

github-actions Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-10-14 03:25 UTC

@ateucher
ateucher requested a review from jules32 October 9, 2026 18:02
@ateucher
ateucher marked this pull request as draft October 9, 2026 18:31
@ateucher
ateucher removed the request for review from jules32 October 9, 2026 18:31
@ateucher
ateucher marked this pull request as ready for review October 9, 2026 19:46
@ateucher
ateucher requested review from jules32, sunnydean and u3ks October 9, 2026 19:46
@jules32

jules32 commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Hi @ateucher ! This is looking great! I love the schedule, nice work!

I read through one time and will review more deeply on each page to suggest edits. A few notes until then (or along with my deeper review):

  • could you please add credit to eScience or Ocean Hackweek, with links where possible on those pages?
  • the quarto format looks a bit off, (ex roles page) with the centering, bullets, and wideness of the lines. Could you play around a bit more?
  • along with the above, would adding a TOC on each page make sense (like the right nav bar on the Cookbook?) Maybe it wouldn't help, but something feels missing....

More review coming, thank you!

@jules32

jules32 commented Oct 11, 2026

Copy link
Copy Markdown
Collaborator

Hi Andy!

I just pushed several small-ish commits; my main thinking was that we we'll have simpler Roles so that we can communicate them more easily, so I have reduced some of the roles and the details.

@jules32 jules32 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hello! I'll approve here with my commits! Thanks Andy!

@ateucher

Copy link
Copy Markdown
Contributor Author

Thanks @jules32 these are great changes! Re the formatting, I liked the centered look on the home page but then was fighting with it everywhere else - I think it's better to let it be justified :)

I'll add the attributions better, and put in the ToCs, then merge

@ateucher

Copy link
Copy Markdown
Contributor Author

@jules32 I have made the changes we talked about, feel free to have another look, or let me know if I should just merge!

@jules32 jules32 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good Andy! Just made one small pluralization I saw, please go ahead and merge!

Comment thread about/roles.qmd

---

## Technical Lead

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
## Technical Lead
## Technical Leads

@ateucher
ateucher merged commit ef88034 into main Oct 14, 2026
1 check passed
@ateucher
ateucher deleted the schedule-roles branch October 14, 2026 18:03
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