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.

Sign up for GitHub

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

Jump to bottom

Sphinx docs #1

Merged
merged 24 commits into from
Feb 1, 2024
Merged

Sphinx docs #1

merged 24 commits into from
Feb 1, 2024

Conversation

GenevieveBuckley
Copy link
Owner

Testing for PR in main napari-animation repository

melissawm and others added 23 commits January 24, 2024 21:28
@melissawm
Documentation set up with Sphinx
838feed
@GenevieveBuckley
Fix ruff CI warnings
98bcfda
@GenevieveBuckley
Add 'doc' setup requirements to development install
c39754e
@GenevieveBuckley
Add unlinked documentation file to ToC
e5425cd
@GenevieveBuckley
Add video scraper for example animation gallery
4ef598c
@GenevieveBuckley
Reformat docs/conf.py file with black
2e73dfa
@GenevieveBuckley
Fix ruff formatting warning
707f3f8
@GenevieveBuckley
Sphinx docs build producing warning where .mov files exist, we'll use…
5efe40f 
… .mp4 instead
@GenevieveBuckley
Make output video filenames match name of script that generated it
1843ede
@GenevieveBuckley
More meaningful name for layer_plane_example.py
1084327
@GenevieveBuckley
Shorter name for layer planes example, fits better in gallery thumbnail
02f6470
@GenevieveBuckley
Add outside link to main napari.org code of conduct page
84993ba
@GenevieveBuckley
Minor text update
9109682
@melissawm
Include README in index and address review comments
58b8ec3
@melissawm
Fix reference in README
a4414a3
@GenevieveBuckley
Add docs build & deploy script, based on napari/docs github actions s…
c9b119d 
…cript
@GenevieveBuckley
Fix HTML formatting of warning/important/note textboxes (breaks githu…
e13f116 
…b markdown rendering of formatting slightly, but better to prioritize the official docs website)
@GenevieveBuckley
Have only one top level heading on docs homepage
5560529
@GenevieveBuckley
No link checking is actually being performed, remove cruft
f2750bd
@GenevieveBuckley
No github hyperlinks are actually being modified, remove this un-used…
d422921 
… piece of code
@GenevieveBuckley
Allow fallback from VideoScraper to static napari screenshot scraper …
2f3a575 
…if no video files found saved to disk
@GenevieveBuckley
Update dependencies
16f5ce3
@GenevieveBuckley
Try github actions permission write contents in yaml script, avoids n…
7824db7 
…eed for separate token
@pep8speaks
Copy link

pep8speaks commented Feb 1, 2024 •
edited
Loading

Hello @GenevieveBuckley! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 161:80: E501 line too long (80 > 79 characters)
Line 167:80: E501 line too long (82 > 79 characters)
Line 180:80: E501 line too long (86 > 79 characters)
Line 192:80: E501 line too long (86 > 79 characters)
Line 210:80: E501 line too long (85 > 79 characters)
Line 212:80: E501 line too long (97 > 79 characters)
Line 281:80: E501 line too long (80 > 79 characters)
Line 285:80: E501 line too long (96 > 79 characters)
Line 293:80: E501 line too long (85 > 79 characters)
Line 295:80: E501 line too long (97 > 79 characters)

Comment last updated at 2024-02-01 01:01:44 UTC

@GenevieveBuckley GenevieveBuckley merged commit ff9d817 into main Feb 1, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@GenevieveBuckley @pep8speaks @melissawm