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 warnings fix #1612

Merged
merged 26 commits into from
Oct 5, 2017
Merged

Sphinx warnings fix #1612

merged 26 commits into from
Oct 5, 2017

Conversation

anotherbugmaster
Copy link
Contributor

Resolved all sphinx warnings except the one related to simserver.

anotherbugmaster and others added 26 commits September 30, 2017 15:39
@anotherbugmaster
Fix typo
b260d4b
@anotherbugmaster
Make save_corpus private
36d98d1
@anotherbugmaster
Annotate bleicorpus.py
981ebbb
@anotherbugmaster
Make __save_corpus weakly private
3428113
@anotherbugmaster
Fix _save_corpus in tests
69fc7e0
@anotherbugmaster
Fix _save_corpus[2]
b65a69a
@anotherbugmaster
Fix relativly obvious sphinx warnings
7fbdf5d
@anotherbugmaster
Fix sphinx warnings
8c00de8
@anotherbugmaster
Revert "Fix sphinx warnings"
ca3d216 
This reverts commit 8c00de8.
@anotherbugmaster
Revert "Fix relativly obvious sphinx warnings"
b4dddb3 
This reverts commit 7fbdf5d.
@anotherbugmaster
Revert "Fix _save_corpus[2]"
1b65791 
This reverts commit b65a69a.
@anotherbugmaster
Revert "Fix _save_corpus in tests"
0b75a9e 
This reverts commit 69fc7e0.
@anotherbugmaster
Revert "Make __save_corpus weakly private"
3a62d72 
This reverts commit 3428113.
@anotherbugmaster
Revert "Annotate bleicorpus.py"
a3d0b6e 
This reverts commit 981ebbb.
@anotherbugmaster
Revert "Make save_corpus private"
7ba8cb3 
This reverts commit 36d98d1.
@anotherbugmaster
Revert "Fix typo"
5fb02bb 
This reverts commit b260d4b.
@anotherbugmaster
Revert "Revert "Fix relativly obvious sphinx warnings""
032480c 
This reverts commit b4dddb3.
@anotherbugmaster
Revert "Revert "Fix sphinx warnings""
51e9f52 
This reverts commit ca3d216.
@menshikh-iv
fix PEP8
e6dc45a
@menshikh-iv
fix last sphinx warnings
359efa2
@menshikh-iv
add missing submodules to reference
23dc197
@menshikh-iv
add missing *.rst
49677df
@menshikh-iv
fix new warnings
9b51aa7
@menshikh-iv
add [docs] deps for building + remove [wmd]
ae0701d
@menshikh-iv
add doc build to travis
8724f65
@menshikh-iv
fix PEP8
593a115
@menshikh-iv
Copy link
Contributor

menshikh-iv commented Oct 5, 2017 •
edited
Loading

What's done:

  • Add missing .rst for submodules & updated apiref
  • Sphinx warning fixes (before and after adding missing .rst)
  • Add building doc to Travis with warning == error option (now build failed if doc is broken, i.e. when sphinx have any problems with building)
  • Add [docs] option in setup.py (for fast installation dependencies for doc building)
  • PEP8 fixes

@menshikh-iv menshikh-iv added the incubator project PR is RaRe incubator project label Oct 5, 2017
@menshikh-iv
Copy link
Contributor

Gratz @anotherbugmaster with your first contribution:1st_place_medal:

@menshikh-iv menshikh-iv merged commit 0ef1ece into piskvorky:develop Oct 5, 2017
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
incubator project PR is RaRe incubator project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@anotherbugmaster @menshikh-iv