Skip to content

Fix some pydocstyle lints#784

Merged
lgarithm merged 31 commits into
tensorlayer:masterfrom
lgarithm:cleanup-docs
Aug 15, 2018
Merged

Fix some pydocstyle lints#784
lgarithm merged 31 commits into
tensorlayer:masterfrom
lgarithm:cleanup-docs

Conversation

@lgarithm
Copy link
Copy Markdown
Member

@lgarithm lgarithm commented Aug 14, 2018

Checklist

  • I've tested that my changes are compatible with the latest version of Tensorflow.
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary.

Motivation and Context

Description

fixed and enforced pydocstyle D210, D200, D301, D207, D403, D204, D412, D402, D300, D208

@tensorlayer tensorlayer deleted a comment Aug 14, 2018
@tensorlayer tensorlayer deleted a comment Aug 14, 2018
@tensorlayer tensorlayer deleted a comment Aug 14, 2018
@tensorlayer tensorlayer deleted a comment Aug 14, 2018
@tensorlayer tensorlayer deleted a comment Aug 14, 2018
@tensorlayer tensorlayer deleted a comment Aug 14, 2018
@lgarithm lgarithm changed the title [WIP] Fix some pydocstyle lints Fix some pydocstyle lints Aug 15, 2018
@lgarithm lgarithm merged commit fd1cf2f into tensorlayer:master Aug 15, 2018
@lgarithm lgarithm deleted the cleanup-docs branch August 15, 2018 06:49
luomai pushed a commit that referenced this pull request Nov 21, 2018
* RTD with Mock Imports

* cleanup docs (fix D208)

* fix D300

* fix D402

* fix D412

* fix D204

* fix D403

* fix D207

* fix D301

* fix D200

* fix format

* disable D302

* fix D210

* fix D202

* fix doc test and make it happen earlier

* Issue with Deprecated Decorator Fixed

* Changelog updated + YAPF cleaning

* Update .readthedocs.yml

* disable D202, since it has conflict with yapf

* TF 1.6.0 Errors fixed

* YAPF Cleaning

* Doc Mock Import for ROI Pooling

* TL init if protected

* Error fix - Test Documentation

* fix documentation test

* update changelog
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.

2 participants