azdev 0.1.92__tar.gz → 0.1.93__tar.gz

{azdev-0.1.92 → azdev-0.1.93}/HISTORY.rst

@@ -2,6 +2,10 @@
 
 Release History
 ===============
+0.1.93
+++++++
+* `azdev linter`: Fix `None` path for added files in `git diff` for `missing_command_example` rule
+
 0.1.92
 ++++++
 * `azdev linter`: Add `missing_command_example` rule

{azdev-0.1.92/azdev.egg-info → azdev-0.1.93}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: azdev
-Version: 0.1.92
+Version: 0.1.93
 Summary: Microsoft Azure CLI Developer Tools
 Home-page: https://github.com/Azure/azure-cli-dev-tools
 Author: Microsoft Corporation
@@ -148,6 +148,10 @@ License
 
 Release History
 ===============
+0.1.93
+++++++
+* `azdev linter`: Fix `None` path for added files in `git diff` for `missing_command_example` rule
+
 0.1.92
 ++++++
 * `azdev linter`: Add `missing_command_example` rule

{azdev-0.1.92 → azdev-0.1.93}/azdev/__init__.py

@@ -4,4 +4,4 @@
 # license information.
 # -----------------------------------------------------------------------------
 
-__VERSION__ = '0.1.92'
+__VERSION__ = '0.1.93'

{azdev-0.1.92 → azdev-0.1.93}/azdev/operations/linter/linter.py

@@ -237,12 +237,13 @@ class Linter:  # pylint: disable=too-many-public-methods, too-many-instance-attr
             cmd_help = self._loaded_help.get(cmd, None)
             if not cmd_help:
                 continue
-            # parameters = cmd_help.parameters
-            # add if future parameter set required
             cmd_suffix = cmd.split()[-1]
             cmd_example_threshold = get_cmd_example_threshold(cmd_suffix, cmd_example_config)
             if cmd_example_threshold == 0:
                 continue
+            if not hasattr(cmd_help, "parameters") or len(cmd_help.parameters) == 0:
+                # skip cmd without parameters
+                continue
             if not hasattr(cmd_help, "examples") or len(cmd_help.examples) < cmd_example_threshold:
                 violations.append(f'Command `{cmd}` should have at least {cmd_example_threshold} example(s)')
         if violations:
@@ -427,7 +428,9 @@ class Linter:  # pylint: disable=too-many-public-methods, too-many-instance-attr
         modified_commands = set()
         diff_patches = diff_branch_file_patch(repo=self.git_repo, target=self.git_target, source=self.git_source)
         for change in diff_patches:
-            file_path, filename = self._split_path(change.a_path)
+            if not change.b_path or not change.diff:
+                continue
+            file_path, filename = self._split_path(change.b_path)
             if "commands.py" not in filename and "aaz" not in file_path:
                 continue
             current_lines = self._read_blob_lines(change.b_blob)
@@ -462,6 +465,8 @@ class Linter:  # pylint: disable=too-many-public-methods, too-many-instance-attr
             return
         diff_patches = diff_branch_file_patch(repo=self.git_repo, target=self.git_target, source=self.git_source)
         for change in diff_patches:
+            if not change.diff:
+                continue
             patch = change.diff.decode("utf-8")
             added_lines = [line for line in patch.splitlines() if line.startswith('+') and not line.startswith('+++')]
             self.diffed_lines |= set(added_lines)

{azdev-0.1.92 → azdev-0.1.93/azdev.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: azdev
-Version: 0.1.92
+Version: 0.1.93
 Summary: Microsoft Azure CLI Developer Tools
 Home-page: https://github.com/Azure/azure-cli-dev-tools
 Author: Microsoft Corporation
@@ -148,6 +148,10 @@ License
 
 Release History
 ===============
+0.1.93
+++++++
+* `azdev linter`: Fix `None` path for added files in `git diff` for `missing_command_example` rule
+
 0.1.92
 ++++++
 * `azdev linter`: Add `missing_command_example` rule