Skip to content

Move authors to frontmatter#266

Merged
bsipocz merged 3 commits intomainfrom
raen/authors-to-frontmatter
Feb 27, 2026
Merged

Move authors to frontmatter#266
bsipocz merged 3 commits intomainfrom
raen/authors-to-frontmatter

Conversation

@troyraen
Copy link
Contributor

This moves the notebook author lists out of the "About this Notebook" sections and into the frontmatter so that they show up under the notebook titles. Also removes the generic "IRSA scientists and developers". We need to do this for the SPHEREx source discovery notebook rather quickly but we want to do it for all notebooks anyway.

It might be good to clean up a few of these author lists but I'll bring that up separately so that it doesn't hold up this PR.

@troyraen troyraen requested a review from bsipocz February 27, 2026 05:10
@troyraen troyraen added documentation Improvements or additions to documentation maintenance General maintenance of the content and/or infrastructure labels Feb 27, 2026
@bsipocz
Copy link
Member

bsipocz commented Feb 27, 2026

While the content looks OK, both circleCI and Github has status issues, so this indeed need to wait until tomorrow -- CI to be restarted in the morning to double check the rendering.

@troyraen
Copy link
Contributor Author

Totally fine to wait until tomorrow.

I think circleci is failing because I touched all notebooks and that causes grep -vf changed_tutorials all_tutorials to return exit code 1, which is treated as a failure. So we probably need to update tox.ini either here or in a separate pr.

@bsipocz
Copy link
Member

bsipocz commented Feb 27, 2026

I think circleci is failing because I touched all notebooks and that causes grep -vf changed_tutorials all_tutorials to return exit code 1, which is treated as a failure. So we probably need to update tox.ini either here or in a separate pr.

Oh, I thought I fixed all of those already, but now I see it's an unmerged changed in an open PR. I'll cherry-pick those changes into a separate PR.

@bsipocz
Copy link
Member

bsipocz commented Feb 27, 2026

Could you please rebase?

@troyraen troyraen force-pushed the raen/authors-to-frontmatter branch from 210b100 to 680cee5 Compare February 27, 2026 09:13
Copy link
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

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

Rendering looks good, and CI failure is unrelated. Thanks!

@bsipocz bsipocz merged commit f8f0ae7 into main Feb 27, 2026
7 of 8 checks passed
@bsipocz bsipocz deleted the raen/authors-to-frontmatter branch February 27, 2026 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation maintenance General maintenance of the content and/or infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants