8000 Changes to validate_docstring script to be able to check all docstrings at once by datapythonista · Pull Request #22408 · pandas-dev/pandas · GitHub
[go: up one dir, main page]
Skip to content

Changes to validate_docstring script to be able to check all docstrings at once #22408

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

Merged
merged 13 commits into from
Oct 13, 2018
Merged

Changes to validate_docstring script to be able to check all docstrings at once #22408

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
b31fc10
Main refactoring of validate_docstrings scripts, so it can be execute…
datapythonista Aug 18, 2018
5d2815f
Fixing errors with introspection
datapythonista Aug 18, 2018
5bb7b9b
Adapting tests to the new format
datapythonista Aug 18, 2018
86327d4
Merge remote-tracking branch 'upstream/master' into validate_all_docs…
datapythonista Aug 20, 2018
188f4a3
Addressing comments from the code review
datapythonista Aug 20, 2018
0d25298
Merge remote-tracking branch 'upstream/master' into validate_all_docs…
datapythonista Aug 21, 2018
577308e
Merge remote-tracking branch 'upstream/master' into validate_all_docs…
datapythonista Sep 1, 2018
156b997
Merging from master
datapythonista Sep 28, 2018
3a5e7fd
Fixing pep8
datapythonista Sep 28, 2018
1a8aee9
Merge branch 'validate_all_docstrings' of github.com:datapythonista/p…
datapythonista Sep 29, 2018
aa4807a
Merge remote-tracking branch 'upstream/master' into validate_all_docs…
datapythonista Sep 29, 2018
1d2c5c0
Adding tests to the function that parses api.rst
datapythonista Sep 29, 2018
db782fd
Addressing comments from review
datapythonista Oct 8, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merging from master
  • Loading branch information
@datapythonista
datapythonista committed Sep 28, 2018
commit 156b997b2ee43b18773485ccdd7dacd326aa213d
6 changes: 3 additions & 3 deletions scripts/tests/test_validate_docstrings.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -531,7 +531,7 @@ def _import_path(self, klass=None, func=None):

@capture_stderr
def test_good_class(self):
errors = validate_one(self._import_path( # noqa: F821
errors = validate_one(self._import_path(
klass='GoodDocStrings'))['errors']
assert isinstance(errors, list)
assert not errors
Expand All @@ -541,14 +541,14 @@ def test_good_class(self):
'plot', 'sample', 'random_letters', 'sample_values', 'head', 'head1',
'contains', 'mode'])
def test_good_functions(self, func):
errors = validate_one(self._import_path( # noqa: F821
errors = validate_one(self._import_path(
klass='GoodDocStrings', func=func))['errors']
assert isinstance(errors, list)
assert not errors

@capture_stderr
def test_bad_class(self):
errors = validate_one(self._import_path( # noqa: F821
errors = validate_one(self._import_path(
klass='BadGenericDocStrings'))['errors']
assert isinstance(errors, list)
assert errors
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.
0