deadcode 2.3.1__py3-none-any.whl → 2.4.0__py3-none-any.whl
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- deadcode/__init__.py +6 -0
- deadcode/actions/fix_or_show_unused_code.py +21 -15
- deadcode/actions/get_unused_names_error_message.py +10 -8
- deadcode/actions/parse_arguments.py +7 -5
- deadcode/data_types.py +2 -1
- deadcode/utils/fix_indent.py +1 -0
- {deadcode-2.3.1.dist-info → deadcode-2.4.0.dist-info}/METADATA +17 -12
- {deadcode-2.3.1.dist-info → deadcode-2.4.0.dist-info}/RECORD +11 -11
- {deadcode-2.3.1.dist-info → deadcode-2.4.0.dist-info}/WHEEL +0 -0
- {deadcode-2.3.1.dist-info → deadcode-2.4.0.dist-info}/entry_points.txt +0 -0
- {deadcode-2.3.1.dist-info → deadcode-2.4.0.dist-info}/licenses/LICENSE +0 -0
deadcode/__init__.py
CHANGED
|
@@ -38,22 +38,28 @@ def fix_or_show_unused_code(unused_items: Iterable[CodeItem], args: Args) -> str
|
|
|
38
38
|
updated_file_content_lines = remove_file_parts_from_content(file_content_lines, unused_file_parts)
|
|
39
39
|
updated_file_content = b''.join(updated_file_content_lines)
|
|
40
40
|
if updated_file_content.strip():
|
|
41
|
-
if
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
41
|
+
if not args.only or _match(filename, args.only):
|
|
42
|
+
if args.dry:
|
|
43
|
+
with open(filename, 'rb') as f:
|
|
44
|
+
filename_bytes = filename.encode()
|
|
45
|
+
diff = diff_bytes(
|
|
46
|
+
unified_diff,
|
|
47
|
+
f.readlines(),
|
|
48
|
+
updated_file_content_lines,
|
|
49
|
+
fromfile=filename_bytes,
|
|
50
|
+
tofile=filename_bytes,
|
|
51
|
+
)
|
|
52
|
+
# TODO: consider printing result instantly to save memory
|
|
53
|
+
result_chunk = b''.join(diff)
|
|
54
|
+
if args.no_color:
|
|
55
|
+
result.append(result_chunk)
|
|
56
|
+
else:
|
|
57
|
+
result.append(add_colors_to_diff(result_chunk))
|
|
52
58
|
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
|
|
56
|
-
|
|
59
|
+
elif args.fix:
|
|
60
|
+
with open(filename, 'wb') as f:
|
|
61
|
+
# TODO: is there a method writelines?
|
|
62
|
+
f.write(updated_file_content)
|
|
57
63
|
else:
|
|
58
64
|
os.remove(filename)
|
|
59
65
|
|
|
@@ -2,6 +2,7 @@ from typing import Iterable, Optional
|
|
|
2
2
|
|
|
3
3
|
from deadcode.data_types import Args
|
|
4
4
|
from deadcode.visitor.code_item import CodeItem
|
|
5
|
+
from deadcode.visitor.ignore import _match
|
|
5
6
|
|
|
6
7
|
|
|
7
8
|
def get_unused_names_error_message(unused_names: Iterable[CodeItem], args: Args) -> Optional[str]:
|
|
@@ -18,16 +19,17 @@ def get_unused_names_error_message(unused_names: Iterable[CodeItem], args: Args)
|
|
|
18
19
|
|
|
19
20
|
messages = []
|
|
20
21
|
for item in unused_names:
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
22
|
+
if not args.only or _match(item.filename, args.only):
|
|
23
|
+
message = f'{item.filename_with_position} \033[91m{item.error_code}\033[0m '
|
|
24
|
+
message += item.message or (
|
|
25
|
+
f"{item.type_.replace('_', ' ').capitalize()} " f"`\033[1m{item.name}\033[0m` " f"is never used"
|
|
26
|
+
)
|
|
27
|
+
if args.no_color:
|
|
28
|
+
message = message.replace('\033[91m', '').replace('\033[1m', '').replace('\033[0m', '')
|
|
29
|
+
messages.append(message)
|
|
28
30
|
|
|
29
31
|
if args.fix:
|
|
30
|
-
message = f"\nRemoved \033[1m{len(
|
|
32
|
+
message = f"\nRemoved \033[1m{len(messages)}\033[0m unused code item{'s' if len(messages) > 1 else ''}!"
|
|
31
33
|
if args.no_color:
|
|
32
34
|
message = message.replace('\x1b[1m', '').replace('\x1b[0m', '')
|
|
33
35
|
messages.append(message)
|
|
@@ -36,9 +36,15 @@ def parse_arguments(args: Optional[List[str]]) -> Args:
|
|
|
36
36
|
parser.add_argument(
|
|
37
37
|
'--dry',
|
|
38
38
|
help='Show changes which would be made in files with --fix option.',
|
|
39
|
+
action='store_true',
|
|
40
|
+
default=False,
|
|
41
|
+
)
|
|
42
|
+
parser.add_argument(
|
|
43
|
+
'--only',
|
|
44
|
+
help='Filenames (or path expressions), that will be reflected in the output and modified.',
|
|
39
45
|
nargs='*',
|
|
40
46
|
action='append',
|
|
41
|
-
default=[
|
|
47
|
+
default=[],
|
|
42
48
|
type=str,
|
|
43
49
|
)
|
|
44
50
|
parser.add_argument(
|
|
@@ -200,10 +206,6 @@ def parse_arguments(args: Optional[List[str]]) -> Args:
|
|
|
200
206
|
if key in parsed_args:
|
|
201
207
|
parsed_args[key].extend(item)
|
|
202
208
|
|
|
203
|
-
# Show changes for only provided files instead of all
|
|
204
|
-
if len(parsed_args['dry']) > 1 or '--dry' not in args:
|
|
205
|
-
parsed_args['dry'].remove('__all_files__')
|
|
206
|
-
|
|
207
209
|
# Do not fix if dry option is provided:
|
|
208
210
|
if parsed_args['dry']:
|
|
209
211
|
parsed_args['fix'] = False
|
deadcode/data_types.py
CHANGED
|
@@ -14,7 +14,8 @@ Pathname = str # Can contain wildewards
|
|
|
14
14
|
class Args:
|
|
15
15
|
fix: bool = False
|
|
16
16
|
verbose: bool = False
|
|
17
|
-
dry:
|
|
17
|
+
dry: bool = False
|
|
18
|
+
only: Iterable[Pathname] = ()
|
|
18
19
|
paths: Iterable[Pathname] = ()
|
|
19
20
|
exclude: Iterable[Pathname] = ()
|
|
20
21
|
ignore_definitions: Iterable[Pathname] = ()
|
deadcode/utils/fix_indent.py
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
Metadata-Version: 2.3
|
|
2
2
|
Name: deadcode
|
|
3
|
-
Version: 2.
|
|
3
|
+
Version: 2.4.0
|
|
4
4
|
Summary: Find and remove dead code.
|
|
5
5
|
Project-URL: homepage, https://github.com/albertas/deadcode
|
|
6
6
|
Project-URL: documentation, https://deadcode.readthedocs.io/
|
|
@@ -11,12 +11,10 @@ Classifier: License :: OSI Approved :: GNU Affero General Public License v3
|
|
|
11
11
|
Classifier: Natural Language :: English
|
|
12
12
|
Classifier: Operating System :: OS Independent
|
|
13
13
|
Classifier: Programming Language :: Python
|
|
14
|
-
Classifier: Programming Language :: Python :: 3.8
|
|
15
|
-
Classifier: Programming Language :: Python :: 3.9
|
|
16
14
|
Classifier: Programming Language :: Python :: 3.10
|
|
17
15
|
Classifier: Programming Language :: Python :: 3.11
|
|
18
16
|
Classifier: Programming Language :: Python :: 3.12
|
|
19
|
-
Requires-Python: >=3.
|
|
17
|
+
Requires-Python: >=3.10
|
|
20
18
|
Requires-Dist: tomli>=2.0.1
|
|
21
19
|
Provides-Extra: test
|
|
22
20
|
Requires-Dist: hatch; extra == 'test'
|
|
@@ -32,9 +30,10 @@ Description-Content-Type: text/markdown
|
|
|
32
30
|
<h2 align="center">Find and Fix Unused Python Code</h2>
|
|
33
31
|
|
|
34
32
|
<p align="center">
|
|
35
|
-
<a href="https://github.com/
|
|
33
|
+
<a href="https://github.com/pre-commit/pre-commit"><img src="https://img.shields.io/badge/pre--commit-enabled-brightgreen?logo=pre-commit" alt="pre-commit" style="max-width:100%;"></a>
|
|
36
34
|
<a href="https://pypi.org/project/deadcode/"><img alt="PyPI" src="https://img.shields.io/pypi/v/deadcode"></a>
|
|
37
35
|
<a href="https://pepy.tech/project/deadcode"><img alt="Downloads" src="https://static.pepy.tech/badge/deadcode"></a>
|
|
36
|
+
<a href="https://github.com/albertas/deadcode/blob/main/LICENSE"><img alt="License: AGPLv3" src="https://raw.githubusercontent.com/albertas/deadcode/main/docs/_static/AGPLv3-license.svg"></a>
|
|
38
37
|
</p>
|
|
39
38
|
|
|
40
39
|
|
|
@@ -56,7 +55,7 @@ deadcode . --fix --dry
|
|
|
56
55
|
|
|
57
56
|
To see suggested fixes only for `foo.py` file:
|
|
58
57
|
```shell
|
|
59
|
-
deadcode . --fix --dry foo.py
|
|
58
|
+
deadcode . --fix --dry --only foo.py
|
|
60
59
|
```
|
|
61
60
|
|
|
62
61
|
To fix:
|
|
@@ -82,7 +81,8 @@ ignore-names-in-files = ["migrations"]
|
|
|
82
81
|
| Option | Type | Meaning |
|
|
83
82
|
|-------------------------------------------|------|----------------------------------------------------------------------|
|
|
84
83
|
|`--fix` | - | Automatically remove detected unused code expressions from the code base. |
|
|
85
|
-
|`--dry` | -
|
|
84
|
+
|`--dry` | - | Show changes which would be made in files. |
|
|
85
|
+
|`--only` | list | Filenames (or path expressions), that will be reflected in the output (and modified if needed). |
|
|
86
86
|
|`--exclude` | list | Filenames (or path expressions), which will be completely skipped without being analysed. |
|
|
87
87
|
|`--ignore-names` | list | Removes provided list of names from the output. Regexp expressions to match multiple names can also be provided, e.g. `*Mixin` will match all classes ending with `Mixin`. |
|
|
88
88
|
|`--ignore-names-in-files` | list | Ignores unused names in files, which filenames match provided path expressions. |
|
|
@@ -100,9 +100,9 @@ ignore-names-in-files = ["migrations"]
|
|
|
100
100
|
|
|
101
101
|
|
|
102
102
|
##### Glossory
|
|
103
|
-
name - variable, function or class name.
|
|
104
|
-
body - code block which follows after `:` in function or class definition.
|
|
105
|
-
definition - whole class or function definition expression including its name and body.
|
|
103
|
+
- `name` - variable, function or class name.
|
|
104
|
+
- `body` - code block which follows after `:` in function or class definition.
|
|
105
|
+
- `definition` - whole class or function definition expression including its name and body.
|
|
106
106
|
|
|
107
107
|
|
|
108
108
|
## Rules
|
|
@@ -185,13 +185,11 @@ code base is implemented in.
|
|
|
185
185
|
- [ ] Distinguish between definitions with same name, but different files.
|
|
186
186
|
- [ ] Repeated application of `deadcode` till the output stops changing.
|
|
187
187
|
- [ ] Unreachable code detection and fixing: this should only be scoped for if statements and only limited to primitive variables.
|
|
188
|
-
- [x] `--fix --dry [filenames]` - only show whats about to change in the listed filenames.
|
|
189
188
|
- [ ] Benchmarking performance with larger projects (time, CPU and memory consumption) in order to optimize.
|
|
190
189
|
- [ ] `--fix` could accept a list of filenames as well (only those files would be changed, but the summary could would be full).
|
|
191
190
|
(This might be confusing, because filenames, which have to be considered are provided without any flag, --fix is expected to not accept arguments)
|
|
192
191
|
- [ ] pre-commit-hook.
|
|
193
192
|
- [ ] language server.
|
|
194
|
-
- [x] Use only two digits for error codes instead of 3. Two is plenty and it simplifies usage a bit
|
|
195
193
|
- [ ] DC10: remove code after terminal statements like `raise`, `return`, `break`, `continue` and comes in the same scope.
|
|
196
194
|
- [ ] Add `ignore` and `per-file-ignores` command line and pyproject.toml options, which allows to skip some rules.
|
|
197
195
|
- [ ] Make sure that all rules are being skipped by `noqa` comment and all rules react to `noqa: rule_id` comments.
|
|
@@ -200,8 +198,15 @@ code base is implemented in.
|
|
|
200
198
|
documentation should cleary demonstrate the behaviour/example that "Schema" means "*.Schema".
|
|
201
199
|
- [ ] Redefinition of an existing name makes previous name unreachable, unless it is assigned somehow.
|
|
202
200
|
- [ ] Check if file is still valid/parsable after automatic fixing, if not: halt the change and report error.
|
|
201
|
+
- [ ] Investigate ways of extracting and backporting Python3.10+ `ast` implementation to lower Python versions.
|
|
203
202
|
|
|
204
203
|
## Release notes
|
|
204
|
+
- v2.4.0:
|
|
205
|
+
- Add `--only` option that accepts filenames only which will be reflected in the output and modified.
|
|
206
|
+
This option can be used with `--fix` and `--fix --dry` options as well as for simple unused code detection without fixing.
|
|
207
|
+
- v2.3.2:
|
|
208
|
+
- Add `pre-commit` hook support.
|
|
209
|
+
- Drop support for Python 3.8 and 3.9 versions, since their `ast` implementation is lacking features.
|
|
205
210
|
- v2.3.1:
|
|
206
211
|
- Started analysing files in bytes instead of trying to convert them into UTF-8 encoded strings.
|
|
207
212
|
- Improved automatic removal of unused imports.
|
|
@@ -1,20 +1,20 @@
|
|
|
1
|
-
deadcode/__init__.py,sha256=
|
|
1
|
+
deadcode/__init__.py,sha256=FVV1tdqGE_2-b70g8FPwZhmFrQ9hCXwuWb7cn9f0w3M,225
|
|
2
2
|
deadcode/cli.py,sha256=F9YO08LvhTAs_UIGklTvILR1Fw3ghDD0gafRy1-7xe8,1175
|
|
3
3
|
deadcode/constants.py,sha256=ya_JsIQGtWAgrEZWc2NpKtuhwkn_05tlGBQpAm3f3Hc,838
|
|
4
|
-
deadcode/data_types.py,sha256=
|
|
4
|
+
deadcode/data_types.py,sha256=Qh7J-7-a-1eVbNLJl8XpOe6G9vIXCZa24BCmA5Jrygo,1337
|
|
5
5
|
deadcode/actions/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
|
|
6
6
|
deadcode/actions/find_python_filenames.py,sha256=uUKvCcisGc4zb6CjUDOhGYPjwW4S2Ysfr2MRhkMD7m0,1546
|
|
7
7
|
deadcode/actions/find_unused_names.py,sha256=TagYxuxNrBarwmJ23tuxuCGNinj8mQo26RGZTh90a-8,494
|
|
8
|
-
deadcode/actions/fix_or_show_unused_code.py,sha256=
|
|
9
|
-
deadcode/actions/get_unused_names_error_message.py,sha256=
|
|
8
|
+
deadcode/actions/fix_or_show_unused_code.py,sha256=7txuOL_4wYiBecSMhGJ15YotNtlTWihzykQOPmTmc5M,2930
|
|
9
|
+
deadcode/actions/get_unused_names_error_message.py,sha256=OMQyCXYTs5FAEFzrcFHZAJ_UrrphbAfn9ahtQG7Nt1I,1285
|
|
10
10
|
deadcode/actions/merge_overlaping_file_parts.py,sha256=ukErzpXCuUOWk5CCpPxD43edqoE087sGJGvYZnwUGXs,3165
|
|
11
11
|
deadcode/actions/parse_abstract_syntax_tree.py,sha256=iL0ukt-FQKA9DuFUk7-sSJZ9nfBzDlJBY-ijItYR6PU,547
|
|
12
|
-
deadcode/actions/parse_arguments.py,sha256=
|
|
12
|
+
deadcode/actions/parse_arguments.py,sha256=xYozzUjDNogVsYAh-0k0zVTauTr8GFWY0bOjTH2skDU,6970
|
|
13
13
|
deadcode/actions/remove_file_parts_from_content.py,sha256=vz9fjniO6Tx3y-rUb4pmAWwU3BYbCBE4-ZY7aDNNnMI,6475
|
|
14
14
|
deadcode/utils/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
|
|
15
15
|
deadcode/utils/add_colors_to_diff.py,sha256=JSEtaHbyojIpCOBa2UNCaKDJr0a9hxkeDlRB4h-YpBI,423
|
|
16
16
|
deadcode/utils/base_test_case.py,sha256=QvUgLgZ_yklDahwO-7StEKxgt12sSbcO3K3-ha4AtQk,3492
|
|
17
|
-
deadcode/utils/fix_indent.py,sha256=
|
|
17
|
+
deadcode/utils/fix_indent.py,sha256=8eApS20lC6i2CwmMpBhTX8iRyJ3yOZJ_QaJVa-cQlfo,2368
|
|
18
18
|
deadcode/utils/flatten_lists.py,sha256=iRGu_oVQZazcAek77QI1FyewxLhP5Ea1AMY_Exs7eHM,616
|
|
19
19
|
deadcode/utils/nested_scopes.py,sha256=jlVrSnDjgeE7Tj5sSANi54woM7Vlr5VNYlHXxX7epFE,3467
|
|
20
20
|
deadcode/utils/print_ast.py,sha256=ovXtdOSgkjQGAeplFt3_LAv3K2r8iiRGu7c6RrflpUU,87
|
|
@@ -25,8 +25,8 @@ deadcode/visitor/ignore.py,sha256=m8qB8uKYB65D-53JOB5LKz32TF9c1y623EyZ1uFFc9A,31
|
|
|
25
25
|
deadcode/visitor/lines.py,sha256=0NCrWAqYua7k93VYy7yLby7olhT2Pe4VU_Z5ImNKkjE,3241
|
|
26
26
|
deadcode/visitor/noqa.py,sha256=0zRA9q0MzEzKe-N285JafP1DSJ3QqGL4GTb9m0QwCiY,2088
|
|
27
27
|
deadcode/visitor/utils.py,sha256=55WcdKJPRrQiFrc0tIkcGO7dUkTnfvZ3qSzsLsTR4MI,2765
|
|
28
|
-
deadcode-2.
|
|
29
|
-
deadcode-2.
|
|
30
|
-
deadcode-2.
|
|
31
|
-
deadcode-2.
|
|
32
|
-
deadcode-2.
|
|
28
|
+
deadcode-2.4.0.dist-info/METADATA,sha256=NT---6Vv93M9jHBL5_nRpx-K2K1mlvlt1sAa1EU03MI,11892
|
|
29
|
+
deadcode-2.4.0.dist-info/WHEEL,sha256=1yFddiXMmvYK7QYTqtRNtX66WJ0Mz8PYEiEUoOUUxRY,87
|
|
30
|
+
deadcode-2.4.0.dist-info/entry_points.txt,sha256=B0nEkycvQHBUJVGS4KmWCBtcWoEM87Gae6zHbrXCU2Q,47
|
|
31
|
+
deadcode-2.4.0.dist-info/licenses/LICENSE,sha256=hIahDEOTzuHCU5J2nd07LWwkLW7Hko4UFO__ffsvB-8,34523
|
|
32
|
+
deadcode-2.4.0.dist-info/RECORD,,
|
|
File without changes
|
|
File without changes
|
|
File without changes
|