deadcode 2.3.2__tar.gz → 2.4.1__tar.gz

{deadcode-2.3.2 → deadcode-2.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: deadcode
-Version: 2.3.2
+Version: 2.4.1
 Summary: Find and remove dead code.
 Project-URL: homepage, https://github.com/albertas/deadcode
 Project-URL: documentation, https://deadcode.readthedocs.io/
@@ -55,7 +55,7 @@ deadcode . --fix --dry
 
 To see suggested fixes only for `foo.py` file:
 ```shell
-deadcode . --fix --dry foo.py
+deadcode . --fix --dry --only foo.py
 ```
 
 To fix:
@@ -81,7 +81,8 @@ ignore-names-in-files = ["migrations"]
 | Option                                    | Type | Meaning  |
 |-------------------------------------------|------|----------------------------------------------------------------------|
 |`--fix`                                    | -    | Automatically remove detected unused code expressions from the code base. |
-|`--dry`                                    | - or list | Show changes which would be made in files. Shows changes for provided filenames or shows all changes if no filename is specified. |
+|`--dry`                                    | -    | Show changes which would be made in files. |
+|`--only`                                   | list | Filenames (or path expressions), that will be reflected in the output (and modified if needed). |
 |`--exclude`                                | list | Filenames (or path expressions), which will be completely skipped without being analysed. |
 |`--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`. |
 |`--ignore-names-in-files`                  | list | Ignores unused names in files, which filenames match provided path expressions. |
@@ -184,13 +185,11 @@ code base is implemented in.
 - [ ] Distinguish between definitions with same name, but different files.
 - [ ] Repeated application of `deadcode` till the output stops changing.
 - [ ] Unreachable code detection and fixing: this should only be scoped for if statements and only limited to primitive variables.
-- [x] `--fix --dry [filenames]` - only show whats about to change in the listed filenames.
 - [ ] Benchmarking performance with larger projects (time, CPU and memory consumption) in order to optimize.
 - [ ] `--fix` could accept a list of filenames as well (only those files would be changed, but the summary could would be full).
     (This might be confusing, because filenames, which have to be considered are provided without any flag, --fix is expected to not accept arguments)
 - [ ] pre-commit-hook.
 - [ ] language server.
-- [x] Use only two digits for error codes instead of 3. Two is plenty and it simplifies usage a bit
 - [ ] DC10: remove code after terminal statements like `raise`, `return`, `break`, `continue` and comes in the same scope.
 - [ ] Add `ignore` and `per-file-ignores` command line and pyproject.toml options, which allows to skip some rules.
 - [ ] Make sure that all rules are being skipped by `noqa` comment and all rules react to `noqa: rule_id` comments.
@@ -199,11 +198,18 @@ code base is implemented in.
   documentation should cleary demonstrate the behaviour/example that "Schema" means "*.Schema".
 - [ ] Redefinition of an existing name makes previous name unreachable, unless it is assigned somehow.
 - [ ] Check if file is still valid/parsable after automatic fixing, if not: halt the change and report error.
+- [ ] Investigate ways of extracting and backporting Python3.10+ `ast` implementation to lower Python versions.
 
 ## Release notes
+- v2.4.1:
+    - Add `--version` option to show `deadcode` version.
+    - Use stdout for `deadcode` output.
+- v2.4.0:
+    - Add `--only` option that accepts filenames only which will be reflected in the output and modified.
+      This option can be used with `--fix` and `--fix --dry` options as well as for simple unused code detection without fixing.
 - v2.3.2:
     - Add `pre-commit` hook support.
-    - Drop support for Python 3.8 and 3.9 versions, since their ast implementation is lacking features.
+    - Drop support for Python 3.8 and 3.9 versions, since their `ast` implementation is lacking features.
 - v2.3.1:
     - Started analysing files in bytes instead of trying to convert them into UTF-8 encoded strings.
     - Improved automatic removal of unused imports.

{deadcode-2.3.2 → deadcode-2.4.1}/README.md

@@ -28,7 +28,7 @@ deadcode . --fix --dry
 
 To see suggested fixes only for `foo.py` file:
 ```shell
-deadcode . --fix --dry foo.py
+deadcode . --fix --dry --only foo.py
 ```
 
 To fix:
@@ -54,7 +54,8 @@ ignore-names-in-files = ["migrations"]
 | Option                                    | Type | Meaning  |
 |-------------------------------------------|------|----------------------------------------------------------------------|
 |`--fix`                                    | -    | Automatically remove detected unused code expressions from the code base. |
-|`--dry`                                    | - or list | Show changes which would be made in files. Shows changes for provided filenames or shows all changes if no filename is specified. |
+|`--dry`                                    | -    | Show changes which would be made in files. |
+|`--only`                                   | list | Filenames (or path expressions), that will be reflected in the output (and modified if needed). |
 |`--exclude`                                | list | Filenames (or path expressions), which will be completely skipped without being analysed. |
 |`--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`. |
 |`--ignore-names-in-files`                  | list | Ignores unused names in files, which filenames match provided path expressions. |
@@ -157,13 +158,11 @@ code base is implemented in.
 - [ ] Distinguish between definitions with same name, but different files.
 - [ ] Repeated application of `deadcode` till the output stops changing.
 - [ ] Unreachable code detection and fixing: this should only be scoped for if statements and only limited to primitive variables.
-- [x] `--fix --dry [filenames]` - only show whats about to change in the listed filenames.
 - [ ] Benchmarking performance with larger projects (time, CPU and memory consumption) in order to optimize.
 - [ ] `--fix` could accept a list of filenames as well (only those files would be changed, but the summary could would be full).
     (This might be confusing, because filenames, which have to be considered are provided without any flag, --fix is expected to not accept arguments)
 - [ ] pre-commit-hook.
 - [ ] language server.
-- [x] Use only two digits for error codes instead of 3. Two is plenty and it simplifies usage a bit
 - [ ] DC10: remove code after terminal statements like `raise`, `return`, `break`, `continue` and comes in the same scope.
 - [ ] Add `ignore` and `per-file-ignores` command line and pyproject.toml options, which allows to skip some rules.
 - [ ] Make sure that all rules are being skipped by `noqa` comment and all rules react to `noqa: rule_id` comments.
@@ -172,11 +171,18 @@ code base is implemented in.
   documentation should cleary demonstrate the behaviour/example that "Schema" means "*.Schema".
 - [ ] Redefinition of an existing name makes previous name unreachable, unless it is assigned somehow.
 - [ ] Check if file is still valid/parsable after automatic fixing, if not: halt the change and report error.
+- [ ] Investigate ways of extracting and backporting Python3.10+ `ast` implementation to lower Python versions.
 
 ## Release notes
+- v2.4.1:
+    - Add `--version` option to show `deadcode` version.
+    - Use stdout for `deadcode` output.
+- v2.4.0:
+    - Add `--only` option that accepts filenames only which will be reflected in the output and modified.
+      This option can be used with `--fix` and `--fix --dry` options as well as for simple unused code detection without fixing.
 - v2.3.2:
     - Add `pre-commit` hook support.
-    - Drop support for Python 3.8 and 3.9 versions, since their ast implementation is lacking features.
+    - Drop support for Python 3.8 and 3.9 versions, since their `ast` implementation is lacking features.
 - v2.3.1:
     - Started analysing files in bytes instead of trying to convert them into UTF-8 encoded strings.
     - Improved automatic removal of unused imports.

deadcode-2.4.1/deadcode/__init__.py

@@ -0,0 +1,8 @@
+try:
+    import importlib.metadata
+
+    __version__ = importlib.metadata.version(__package__ or __name__)
+except ImportError:
+    import importlib_metadata
+
+    __version__ = importlib_metadata.version(__package__ or __name__)

{deadcode-2.3.2 → deadcode-2.4.1}/deadcode/actions/fix_or_show_unused_code.py

@@ -38,27 +38,28 @@ def fix_or_show_unused_code(unused_items: Iterable[CodeItem], args: Args) -> str
         updated_file_content_lines = remove_file_parts_from_content(file_content_lines, unused_file_parts)
         updated_file_content = b''.join(updated_file_content_lines)
         if updated_file_content.strip():
-            if args.dry and ('__all_files__' in args.dry or _match(filename, args.dry)):
-                with open(filename, 'rb') as f:
-                    filename_bytes = filename.encode()
-                    diff = diff_bytes(
-                        unified_diff,
-                        f.readlines(),
-                        updated_file_content_lines,
-                        fromfile=filename_bytes,
-                        tofile=filename_bytes,
-                    )
-                    # TODO: consider printing result instantly to save memory
-                    result_chunk = b''.join(diff)
-                    if args.no_color:
-                        result.append(result_chunk)
-                    else:
-                        result.append(add_colors_to_diff(result_chunk))
+            if not args.only or _match(filename, args.only):
+                if args.dry:
+                    with open(filename, 'rb') as f:
+                        filename_bytes = filename.encode()
+                        diff = diff_bytes(
+                            unified_diff,
+                            f.readlines(),
+                            updated_file_content_lines,
+                            fromfile=filename_bytes,
+                            tofile=filename_bytes,
+                        )
+                        # TODO: consider printing result instantly to save memory
+                        result_chunk = b''.join(diff)
+                        if args.no_color:
+                            result.append(result_chunk)
+                        else:
+                            result.append(add_colors_to_diff(result_chunk))
 
-            elif args.fix:
-                with open(filename, 'wb') as f:
-                    # TODO: is there a method writelines?
-                    f.write(updated_file_content)
+                elif args.fix:
+                    with open(filename, 'wb') as f:
+                        # TODO: is there a method writelines?
+                        f.write(updated_file_content)
         else:
             os.remove(filename)
 

deadcode-2.4.1/deadcode/actions/get_unused_names_error_message.py

@@ -0,0 +1,37 @@
+from typing import Iterable, Optional
+
+from deadcode.data_types import Args
+from deadcode.visitor.code_item import CodeItem
+from deadcode.visitor.ignore import _match
+
+
+def get_unused_names_error_message(unused_names: Iterable[CodeItem], args: Args) -> Optional[str]:
+    unused_names = list(unused_names)
+
+    if not unused_names:
+        return None
+
+    if args.quiet:
+        return ''
+
+    if args.count:
+        return f'{len(unused_names)}'
+
+    messages = []
+    for item in unused_names:
+        if not args.only or _match(item.filename, args.only):
+            message = f'{item.filename_with_position} \033[91m{item.error_code}\033[0m '
+            message += item.message or (
+                f"{item.type_.replace('_', ' ').capitalize()} " f"`\033[1m{item.name}\033[0m` " f"is never used"
+            )
+            if args.no_color:
+                message = message.replace('\033[91m', '').replace('\033[1m', '').replace('\033[0m', '')
+            messages.append(message)
+
+    if args.fix:
+        message = f"\nRemoved \033[1m{len(messages)}\033[0m unused code item{'s' if len(messages) > 1 else ''}!"
+        if args.no_color:
+            message = message.replace('\x1b[1m', '').replace('\x1b[0m', '')
+        messages.append(message)
+
+    return '\n'.join(messages)

{deadcode-2.3.2 → deadcode-2.4.1}/deadcode/actions/parse_arguments.py

@@ -36,9 +36,15 @@ def parse_arguments(args: Optional[List[str]]) -> Args:
     parser.add_argument(
         '--dry',
         help='Show changes which would be made in files with --fix option.',
+        action='store_true',
+        default=False,
+    )
+    parser.add_argument(
+        '--only',
+        help='Filenames (or path expressions), that will be reflected in the output and modified.',
         nargs='*',
         action='append',
-        default=[['__all_files__']],
+        default=[],
         type=str,
     )
     parser.add_argument(
@@ -188,6 +194,12 @@ def parse_arguments(args: Optional[List[str]]) -> Args:
         action='store_true',
         default=False,
     )
+    parser.add_argument(
+        '--version',
+        help='Shows deadcode version',
+        action='store_true',
+        default=False,
+    )
 
     parsed_args = parser.parse_args(args).__dict__
 
@@ -200,10 +212,6 @@ def parse_arguments(args: Optional[List[str]]) -> Args:
         if key in parsed_args:
             parsed_args[key].extend(item)
 
-    # Show changes for only provided files instead of all
-    if len(parsed_args['dry']) > 1 or '--dry' not in args:
-        parsed_args['dry'].remove('__all_files__')
-
     # Do not fix if dry option is provided:
     if parsed_args['dry']:
         parsed_args['fix'] = False

{deadcode-2.3.2 → deadcode-2.4.1}/deadcode/cli.py

@@ -1,5 +1,7 @@
 from typing import List, Optional
+import sys
 
+from deadcode import __version__
 from deadcode.actions.find_python_filenames import find_python_filenames
 from deadcode.actions.find_unused_names import find_unused_names
 from deadcode.actions.fix_or_show_unused_code import fix_or_show_unused_code
@@ -12,6 +14,10 @@ from deadcode.actions.get_unused_names_error_message import (
 def main(
     command_line_args: Optional[List[str]] = None,
 ) -> Optional[str]:
+
+    if command_line_args and '--version' in command_line_args or '--version' in sys.argv:
+        return __version__
+
     args = parse_arguments(command_line_args)
 
     filenames = find_python_filenames(args=args)
@@ -31,5 +37,10 @@ def main(
     return None
 
 
+def print_main() -> None:
+    if result := main():
+        print(result)
+
+
 if __name__ == '__main__':
-    main()
+    print_main()

{deadcode-2.3.2 → deadcode-2.4.1}/deadcode/data_types.py

@@ -14,7 +14,9 @@ Pathname = str  # Can contain wildewards
 class Args:
     fix: bool = False
     verbose: bool = False
-    dry: Iterable[Pathname] = ()
+    version: bool = False
+    dry: bool = False
+    only: Iterable[Pathname] = ()
     paths: Iterable[Pathname] = ()
     exclude: Iterable[Pathname] = ()
     ignore_definitions: Iterable[Pathname] = ()

{deadcode-2.3.2 → deadcode-2.4.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "deadcode"
-version = "2.3.2"
+version = "2.4.1"
 authors = [
     {name = "Albertas Gimbutas", email = "albertasgim@gmail.com"},
 ]
@@ -33,7 +33,7 @@ test = [
 ]
 
 [project.scripts]
-deadcode = "deadcode.cli:main"
+deadcode = "deadcode.cli:print_main"
 
 [project.urls]
 homepage = "https://github.com/albertas/deadcode"

deadcode-2.3.2/deadcode/actions/get_unused_names_error_message.py

@@ -1,35 +0,0 @@
-from typing import Iterable, Optional
-
-from deadcode.data_types import Args
-from deadcode.visitor.code_item import CodeItem
-
-
-def get_unused_names_error_message(unused_names: Iterable[CodeItem], args: Args) -> Optional[str]:
-    unused_names = list(unused_names)
-
-    if not unused_names:
-        return None
-
-    if args.quiet:
-        return ''
-
-    if args.count:
-        return f'{len(unused_names)}'
-
-    messages = []
-    for item in unused_names:
-        message = f'{item.filename_with_position} \033[91m{item.error_code}\033[0m '
-        message += item.message or (
-            f"{item.type_.replace('_', ' ').capitalize()} " f"`\033[1m{item.name}\033[0m` " f"is never used"
-        )
-        if args.no_color:
-            message = message.replace('\033[91m', '').replace('\033[1m', '').replace('\033[0m', '')
-        messages.append(message)
-
-    if args.fix:
-        message = f"\nRemoved \033[1m{len(unused_names)}\033[0m unused code item{'s' if len(unused_names) > 1 else ''}!"
-        if args.no_color:
-            message = message.replace('\x1b[1m', '').replace('\x1b[0m', '')
-        messages.append(message)
-
-    return '\n'.join(messages)