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

Rename pep tools #29

Merged
merged 16 commits into from Oct 22, 2018
Merged

Rename pep tools #29

merged 16 commits into from Oct 22, 2018

Conversation

tdenewiler
Copy link
Contributor

@creffett, this addresses #28.

I renamed the tools and added warnings about the usage of the old names. If you want to see the warnings you can edit statick_tool/rsc/config.yaml for the self_check level to add in pep257 and pep8. Then run from this branch and do:

./statick . statick_output/ --profile self_check.yaml

You should see some of the output show:

DEPRECATION WARNING: The pep257 tool has been renamed as the pydocstyle tool. Please update your configuration.
Running pydocstyle tool plugin...
DEPRECATION WARNING: The tool name changed from pep257 to pydocstyle. Please update your configuration file to use the new tool name.
pydocstyle tool plugin done.

@tdenewiler
v0.1.3
99c4334
@tdenewiler
v0.1.4
04221bc
@tdenewiler
Merge remote-tracking branch 'upstream/master'
3425276
@tdenewiler
Merge remote-tracking branch 'upstream/master'
e031e03
@tdenewiler
Merge remote-tracking branch 'upstream/master'
fc059f7
@tdenewiler
Merge remote-tracking branch 'upstream/master'
c2fcbac
@tdenewiler
Merge remote-tracking branch 'upstream/master'
d31b083
@tdenewiler
Merge remote-tracking branch 'upstream/master'
d33b818
@tdenewiler
Merge remote-tracking branch 'upstream/master'
8542c15
@tdenewiler
Merge remote-tracking branch 'upstream/master'
a8051a7
@tdenewiler
Merge remote-tracking branch 'upstream/master'
bd07b4b
@tdenewiler
Merge remote-tracking branch 'upstream/master'
6d21250
@tdenewiler
Renamed pep8 and pep257 tools to pycodestyle and pydocstyle. Added de…
e8da08a 
…precation warnings for the previous names when used as tools and in configuration files. This is following upstream changes to the names of the tools themselves.
@tdenewiler
Using existing requirements.txt to install dependencies in Travis. Th…
e3f3c84 
…is is to pick up any new packages that are added and necessary for testing.
@tdenewiler
Updating tool names for documentation files.
49246a5
@tdenewiler
Fixing documentation setup file for travis-sphinx formatting rules.
1a2226d
@tdenewiler
Copy link
Contributor Author

@creffett, I'll wait to merge this until you let me know if this is what you had in mind.

creffett
creffett approved these changes Oct 21, 2018
View reviewed changes
Copy link
Contributor

@creffett creffett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks!

@denewiler denewiler merged commit 9363d1a into sscpac:master Oct 22, 2018
@denewiler denewiler mentioned this pull request Oct 22, 2018
Closed
axydes pushed a commit to axydes/statick that referenced this pull request Jan 29, 2020
@tdenewiler @denewiler
Rename pep tools (sscpac#29)
e9910b8 
* Renamed pep8 and pep257 tools to pycodestyle and pydocstyle. Added deprecation warnings for the previous names when used as tools and in configuration files. This is following upstream changes to the names of the tools themselves.

* Using existing requirements.txt to install dependencies in Travis. This is to pick up any new packages that are added and necessary for testing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@creffett creffett creffett approved these changes

@denewiler denewiler denewiler approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@tdenewiler @creffett @denewiler