Skip to content

remove archived notebooks from the published book - #151

Merged
MathewBiddle merged 3 commits into
ioos:mainfrom
ocefpaf:archive_stale_notebooks
Aug 31, 2023
Merged

remove archived notebooks from the published book#151
MathewBiddle merged 3 commits into
ioos:mainfrom
ocefpaf:archive_stale_notebooks

Conversation

@ocefpaf

@ocefpaf ocefpaf commented Aug 30, 2023

Copy link
Copy Markdown
Member

@MathewBiddle I moved all the stale notebooks to "archived" so they don't get published anymore. Most of them are not worth revisiting but those that are, we still have the file in the repo as a starting point.

@MathewBiddle

Copy link
Copy Markdown
Contributor

ah. Changed the extension, nice. When merged, we should check that the chapter table of contents reflect the change. For example, https://ioos.github.io/ioos_code_lab/content/code_gallery/data_access.html

@MathewBiddle

Copy link
Copy Markdown
Contributor

it would be nice if you could view those notebooks in the web interface. But, this works too.

https://github.com/ocefpaf/ioos_code_lab/blob/archive_stale_notebooks/jupyterbook/content/code_gallery/data_access_notebooks/2016-12-19-exploring_csw.ipynb.archived

@ocefpaf

ocefpaf commented Aug 30, 2023

Copy link
Copy Markdown
Member Author

it would be nice if you could view those notebooks in the web interface

jupyterbook probably has something to ignore some notebooks in some list... Let me check if that exists before we merge this one so we don't need to rename them.

Comment thread jupyterbook/_config.yml
@ocefpaf
ocefpaf requested a review from MathewBiddle August 30, 2023 21:22
@ocefpaf

ocefpaf commented Aug 30, 2023

Copy link
Copy Markdown
Member Author

This one should be ready to go. I'll fix the pre-commits and failing notebooks in #152 to avoid conflicts. Most of the failing notebooks are ERDDAP server failures that may require replacement.

@MathewBiddle

Copy link
Copy Markdown
Contributor

Built the book locally and everything seemed to work as expected. TOC for chapter headers were automatically updated as well. Thanks for addressing this so quickly.

@MathewBiddle
MathewBiddle merged commit a615c54 into ioos:main Aug 31, 2023

@MathewBiddle MathewBiddle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good

@ocefpaf
ocefpaf deleted the archive_stale_notebooks branch August 31, 2023 13:23
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