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

Move to Jupterbook based docs #63

Merged
merged 7 commits into from
Dec 23, 2021
Merged

Move to Jupterbook based docs #63

merged 7 commits into from
Dec 23, 2021

Conversation

mgrover1
Copy link
Contributor

@mgrover1 mgrover1 commented Dec 23, 2021

Change Summary

Migrates from regular sphinx to JupyterBook based documentation

Related issue number

Checklist

  • Unit tests for the changes exist
  • Tests pass on CI
  • Documentation reflects the changes where applicable

@mgrover1
Copy link
Contributor Author

mgrover1 commented Dec 23, 2021

I keep running into the error .pre-commit-hooks.yaml does not exist

@andersy005 have you run across this before?

@andersy005
Copy link
Contributor

@mgrover1, what version of pre-commit are you running?

@mgrover1
Copy link
Contributor Author

@mgrover1, what version of pre-commit are you running?

pre-commit 2.13.0

@andersy005
Copy link
Contributor

I'm on pre-commit 2.16.0 and I don't get that error...

@andersy005
Copy link
Contributor

@mgrover1, I push a few changes on your branch...

@andersy005
Copy link
Contributor

@mgrover1, the pre-commit issue should be fixed...

@mgrover1 mgrover1 merged commit 98676b5 into NCAR:main Dec 23, 2021
@andersy005 andersy005 added the documentation Improvements or additions to documentation label Dec 23, 2021
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Documentation
2 participants