Merge ~cjwatson/lazr.config:pre-commit-update into lazr.config:main

Proposed by Colin Watson
Status: Merged
Merged at revision: 9cac7f805424b3b540b37dd426b498af720f2433
Proposed branch: ~cjwatson/lazr.config:pre-commit-update
Merge into: lazr.config:main
Diff against target: 182 lines (+27/-23)
5 files modified
.pre-commit-config.yaml (+5/-5)
setup.py (+1/-1)
src/lazr/config/__init__.py (+3/-4)
src/lazr/config/docs/conf.py (+16/-13)
tox.ini (+2/-0)
Reviewer Review Type Date Requested Status
Jürgen Gmach Approve
Review via email: mp+436528@code.launchpad.net

Commit message

Update pre-commit hooks

Description of the change

To post a comment you must log in.
Revision history for this message
Jürgen Gmach (jugmac00) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
2index 91f3f79..6382dc7 100644
3--- a/.pre-commit-config.yaml
4+++ b/.pre-commit-config.yaml
5@@ -2,7 +2,7 @@
6 # See https://pre-commit.com/hooks.html for more hooks
7 repos:
8 - repo: https://github.com/pre-commit/pre-commit-hooks
9- rev: v4.0.1
10+ rev: v4.4.0
11 hooks:
12 - id: check-added-large-files
13 - id: check-ast
14@@ -13,20 +13,20 @@ repos:
15 - id: check-yaml
16 - id: debug-statements
17 - repo: https://github.com/psf/black
18- rev: 22.3.0
19+ rev: 22.12.0
20 hooks:
21 - id: black
22 # skip namespace package declaration
23 exclude: ^src/lazr/__init__\.py$
24 - repo: https://github.com/PyCQA/isort
25- rev: 5.9.2
26+ rev: 5.12.0
27 hooks:
28 - id: isort
29 - repo: https://github.com/PyCQA/flake8
30- rev: 4.0.1
31+ rev: 6.0.0
32 hooks:
33 - id: flake8
34 - repo: https://github.com/get-woke/woke
35- rev: ee781d3ce0ddf835267764f27f4ffdd2dd21fa27
36+ rev: v0.19.0
37 hooks:
38 - id: woke-from-source
39diff --git a/setup.py b/setup.py
40index 2deb1ba..5448a20 100755
41--- a/setup.py
42+++ b/setup.py
43@@ -58,7 +58,7 @@ files. The format supports schema validation.
44 classifiers=[
45 "Development Status :: 5 - Production/Stable",
46 "Intended Audience :: Developers",
47- "License :: OSI Approved :: GNU Library or Lesser General Public License (LGPL)",
48+ "License :: OSI Approved :: GNU Library or Lesser General Public License (LGPL)", # noqa
49 "Operating System :: OS Independent",
50 "Programming Language :: Python",
51 "Programming Language :: Python :: 2.7",
52diff --git a/src/lazr/config/__init__.py b/src/lazr/config/__init__.py
53index 4a42184..a4dac9c 100644
54--- a/src/lazr/config/__init__.py
55+++ b/src/lazr/config/__init__.py
56@@ -16,12 +16,11 @@
57
58 """A configuration file system."""
59
60-from lazr.config._version import __version__
61-
62-__version__
63-
64 # While we generally frown on "*" imports, this, combined with the fact we
65 # only test code from this module, means that we can verify what has been
66 # exported.
67 from lazr.config._config import * # noqa: F401, F403
68 from lazr.config._config import __all__ # noqa: F401, F402
69+from lazr.config._version import __version__
70+
71+__version__
72diff --git a/src/lazr/config/docs/conf.py b/src/lazr/config/docs/conf.py
73index dd4be76..f9eabe9 100644
74--- a/src/lazr/config/docs/conf.py
75+++ b/src/lazr/config/docs/conf.py
76@@ -3,7 +3,8 @@
77 # lazr.config documentation build configuration file, created by
78 # sphinx-quickstart on Mon Jan 7 10:37:37 2013.
79 #
80-# This file is execfile()d with the current directory set to its containing dir.
81+# This file is execfile()d with the current directory set to its containing
82+# dir.
83 #
84 # Note that not all possible configuration values are present in this
85 # autogenerated file.
86@@ -16,15 +17,15 @@
87 # documentation root, use os.path.abspath to make it absolute, like shown here.
88 # sys.path.insert(0, os.path.abspath('.'))
89
90-# -- General configuration -----------------------------------------------------
91+# -- General configuration ----------------------------------------------------
92
93 from lazr.config import __version__
94
95 # If your documentation needs a minimal Sphinx version, state it here.
96 # needs_sphinx = '1.0'
97
98-# Add any Sphinx extension module names here, as strings. They can be extensions
99-# coming with Sphinx (named 'sphinx.ext.*') or your custom ones.
100+# Add any Sphinx extension module names here, as strings. They can be
101+# extensions coming with Sphinx (named 'sphinx.ext.*') or your custom ones.
102 extensions = []
103
104 # Add any paths that contain templates here, relative to this directory.
105@@ -66,7 +67,8 @@ release = version
106 # directories to ignore when looking for source files.
107 exclude_patterns = ["_build", "eggs"]
108
109-# The reST default role (used for this markup: `text`) to use for all documents.
110+# The reST default role (used for this markup: `text`) to use for all
111+# documents.
112 # default_role = None
113
114 # If true, '()' will be appended to :func: etc. cross-reference text.
115@@ -87,7 +89,7 @@ pygments_style = "sphinx"
116 # modindex_common_prefix = []
117
118
119-# -- Options for HTML output ---------------------------------------------------
120+# -- Options for HTML output --------------------------------------------------
121
122 # The theme to use for HTML and HTML Help pages. See the documentation for
123 # a list of builtin themes.
124@@ -176,21 +178,22 @@ html_sidebars = {
125 htmlhelp_basename = "lazrconfigdoc"
126
127
128-# -- Options for LaTeX output --------------------------------------------------
129+# -- Options for LaTeX output -------------------------------------------------
130
131 # latex_elements = {
132 # The paper size ('letterpaper' or 'a4paper').
133-#'papersize': 'letterpaper',
134+# 'papersize': 'letterpaper',
135
136 # The font size ('10pt', '11pt' or '12pt').
137-#'pointsize': '10pt',
138+# 'pointsize': '10pt',
139
140 # Additional stuff for the LaTeX preamble.
141-#'preamble': '',
142+# 'preamble': '',
143 # }
144
145 # Grouping the document tree into LaTeX files. List of tuples
146-# (source start file, target name, title, author, documentclass [howto/manual]).
147+# (source start file, target name, title, author, documentclass
148+# [howto/manual]).
149
150 # The name of an image file (relative to this directory) to place at the top of
151 # the title page.
152@@ -213,7 +216,7 @@ htmlhelp_basename = "lazrconfigdoc"
153 # latex_domain_indices = True
154
155
156-# -- Options for manual page output --------------------------------------------
157+# -- Options for manual page output -------------------------------------------
158
159 # One entry per manual page. List of tuples
160 # (source start file, name, description, authors, manual section).
161@@ -222,7 +225,7 @@ htmlhelp_basename = "lazrconfigdoc"
162 # man_show_urls = False
163
164
165-# -- Options for Texinfo output ------------------------------------------------
166+# -- Options for Texinfo output -----------------------------------------------
167
168 # Grouping the document tree into Texinfo files. List of tuples
169 # (source start file, target name, title, author,
170diff --git a/tox.ini b/tox.ini
171index 45e4d91..186f2bd 100644
172--- a/tox.ini
173+++ b/tox.ini
174@@ -19,6 +19,8 @@ commands =
175 zope-testrunner --test-path src --tests-pattern ^tests {posargs}
176
177 [testenv:pre-commit]
178+# necessary to build the woke linter
179+passenv = HOME
180 commands =
181 pre-commit run --all-files
182 deps =

Subscribers

People subscribed via source and target branches