Skip to content

Change __all__ to tuples - #97

Merged
white-gecko merged 1 commit into
feature/provenancefrom
feature/__all__Tuples
Jan 5, 2018
Merged

Change __all__ to tuples#97
white-gecko merged 1 commit into
feature/provenancefrom
feature/__all__Tuples

Conversation

@white-gecko

Copy link
Copy Markdown
Member

To prevent the warning:

WARNING: all is defined as a list, this means pydocstyle cannot reliably detect contents of the all variable, because it can be mutated. Change all to be an (immutable) tuple, to remove this warning. Note, pydocstyle uses all to detect which definitions are public, to warn if public definitions are missing docstrings. If all is a (mutable) list, pydocstyle cannot reliably assume its contents. pydocstyle will proceed assuming all is not mutated.

the __all__ "variables" are changed to tuples.

@white-gecko
white-gecko merged commit 98044da into feature/provenance Jan 5, 2018
@white-gecko
white-gecko deleted the feature/__all__Tuples branch January 5, 2018 13:54
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 60.352% when pulling 61cc15d on feature/__all__Tuples into cf12453 on feature/provenance.

1 similar comment
@coveralls

coveralls commented Jan 5, 2018

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 60.352% when pulling 61cc15d on feature/__all__Tuples into cf12453 on feature/provenance.

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.

3 participants