-
Notifications
You must be signed in to change notification settings - Fork 17
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
DOC: re-strucuture the doc folder #514
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #514 +/- ##
=======================================
Coverage 79.31% 79.31%
=======================================
Files 27 27
Lines 3834 3834
=======================================
Hits 3041 3041
Misses 793 793
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. 📢 Have feedback on the report? Share it here. |
d389ebf
to
9340c59
Compare
add logo use pydata css update readthedocs.yml fix-fix Add strict failed when warnings bump optimade-client version Resolve warnings in doc bulid Solve dependencies
Looks better now https://aiidalab-widgets-base--514.org.readthedocs.build/en/514/ |
docs/source/conf.py
Outdated
@@ -46,7 +41,7 @@ | |||
|
|||
# General information about the project. | |||
project = "aiidalab-widgets-base" | |||
copyright_first_year = "2020" | |||
copyright_first_year = "2023" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't it be the year the documentation was created?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are correct, it didn't notice "first_year" 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very minor question, otherwise good 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I actually need those changes very much, because I run into the same issues that you fixed here. So let's merge it 😎
Thanks, @unkcpz!
No description provided.