Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(docs): Update dataset_discovery.ipynb #1006

Merged
merged 1 commit into from Jan 22, 2024
Merged

fix(docs): Update dataset_discovery.ipynb #1006

merged 1 commit into from Jan 22, 2024

Conversation

nsmith-
Copy link
Member

@nsmith- nsmith- commented Jan 22, 2024

The headings for the sub-sections of this notebook are the same level as the overall heading, which breaks the index in the rendered docs:
Screenshot 2024-01-22 at 3 24 19 PM

This should fix it

The headings for the sub-sections of this notebook are the same level as the overall heading, which breaks the index in the rendered docs. This should fix it
@nsmith-
Copy link
Member Author

nsmith- commented Jan 22, 2024

FYI @valsdav

@nsmith- nsmith- changed the title Update dataset_discovery.ipynb [docs] Update dataset_discovery.ipynb Jan 22, 2024
@nsmith- nsmith- changed the title [docs] Update dataset_discovery.ipynb fix(docs) Update dataset_discovery.ipynb Jan 22, 2024
@lgray lgray changed the title fix(docs) Update dataset_discovery.ipynb fix(docs): Update dataset_discovery.ipynb Jan 22, 2024
@lgray
Copy link
Collaborator

lgray commented Jan 22, 2024

@nsmith- did you build the docs locally to check?

@nsmith-
Copy link
Member Author

nsmith- commented Jan 22, 2024

No, but I use subheadings in some of the other notebooks I've made in there.

@lgray lgray merged commit 66bf78e into master Jan 22, 2024
15 of 18 checks passed
@lgray lgray deleted the nsmith--patch-1 branch January 22, 2024 23:15
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.

None yet

2 participants