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.

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

Whitespace hooks are still useful for non-python #816

Merged
merged 1 commit into from
May 14, 2018
Merged

Whitespace hooks are still useful for non-python #816

merged 1 commit into from
May 14, 2018

Conversation

asottile
Copy link
Contributor

This was removed in #813 but there are still lots of source files for which these hooks are still relevant:

.codecov.yml
.github/ISSUE_TEMPLATE.md
.github/PULL_REQUEST_TEMPLATE.md
.gitignore
.pre-commit-config.yaml
.travis-osx
.travis.yml
CHANGELOG.rst
CONTRIBUTING.rst
CONTRIBUTORS
HOWTORELEASE.rst
LICENSE
MANIFEST.in
README.rst
Vagrantfile
appveyor.yml
changelog/.gitkeep
changelog/706.bugfix.rst
changelog/754.misc.rst
changelog/794.feature.rst
changelog/797.doc.rst
changelog/798.feature.rst
changelog/799.doc.rst
changelog/800.misc.rst
changelog/801.misc.rst
changelog/802.misc.rst
changelog/examples.rst
changelog/template.jinja2
doc/Makefile
doc/_static/sphinxdoc.css
doc/_templates/localtoc.html
doc/announce/changelog-only.rst
doc/changelog.rst
doc/config.rst
doc/developers.rst
doc/drafts/extend-envs-and-packagebuilds.md
doc/drafts/tox_conda_notes_niccodemus.md
doc/example/basic.rst
doc/example/devenv.rst
doc/example/general.rst
doc/example/jenkins.rst
doc/example/nose.rst
doc/example/platform.rst
doc/example/pytest.rst
doc/example/result.rst
doc/example/unittest.rst
doc/examples.rst
doc/img/tox.svg
doc/index.rst
doc/install.rst
doc/links.rst
doc/plugins.rst
doc/support.rst
pyproject.toml
setup.cfg
tasks/pra.sh
tox.ini

@gaborbernat
Copy link
Member

@asottile agreed

@codecov
Copy link

codecov bot commented May 14, 2018

Codecov Report

Merging #816 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #816   +/-   ##
=====================================
  Coverage      95%    95%           
=====================================
  Files          11     11           
  Lines        2323   2323           
=====================================
  Hits         2215   2215           
  Misses        108    108

@gaborbernat gaborbernat merged commit 3210b2e into master May 14, 2018
@gaborbernat gaborbernat deleted the ws branch May 14, 2018 02:27
@obestwalter
Copy link
Member

Good point.

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