Conversation
|
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. |
|
Totally fine to wait until tomorrow. I think circleci is failing because I touched all notebooks and that causes |
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. |
|
Could you please rebase? |
210b100 to
680cee5
Compare
bsipocz
left a comment
There was a problem hiding this comment.
Rendering looks good, and CI failure is unrelated. Thanks!
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.