Skip to content

{Pylint} Fix unrecognized-inline-option #18048

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.

Sign up for GitHub

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

Merged
jiasli merged 2 commits into from
May 17, 2021
Merged

{Pylint} Fix unrecognized-inline-option #18048

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1c89789
Fix unrecognized-inline-option
jiasli May 12, 2021
3145eb7
Merge branch 'dev' into unrecognized-inline-option
jiasli May 13, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion pylintrc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ disable=
too-many-function-args,
too-many-lines,
unnecessary-comprehension,
unrecognized-inline-option,
use-a-generator,
using-constant-test,
wrong-import-order,
Expand Down
2 changes: 1 addition & 1 deletion src/azure-cli-core/azure/cli/core/parser.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -284,7 +284,7 @@ def parse_known_args(self, args=None, namespace=None):
self._namespace, self._raw_arguments = super().parse_known_args(args=args, namespace=namespace)
return self._namespace, self._raw_arguments

def _check_value(self, action, value): # pylint: too-many-locals, too-many-branches
def _check_value(self, action, value):
# Override to customize the error message when a argument is not among the available choices
# converted value must be one of the choices (if specified)
if action.choices is not None and value not in action.choices: # pylint: disable=too-many-nested-blocks
Expand Down
2 changes: 1 addition & 1 deletion src/azure-cli/azure/cli/command_modules/eventgrid/custom.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -994,7 +994,7 @@ def _get_event_subscription_info( # pylint: disable=too-many-locals,too-many-

condition1 = endpoint_type is not None and endpoint_type.lower() == WEBHOOK_DESTINATION.lower()
condition2 = delivery_identity_endpoint_type is not None and \
delivery_identity_endpoint_type.lower() == WEBHOOK_DESTINATION.lower() # pylint: line-too-long
delivery_identity_endpoint_type.lower() == WEBHOOK_DESTINATION.lower()
if condition1 or condition2:
tennant_id = azure_active_directory_tenant_id
application_id = azure_active_directory_application_id_or_uri
Expand Down
4 changes: 0 additions & 4 deletions src/azure-cli/azure/cli/command_modules/resource/custom.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1627,7 +1627,6 @@ def _single_or_collection(obj, default=None):
return obj


# pylint: unused-argument
def show_resource(cmd, resource_ids=None, resource_group_name=None,
resource_provider_namespace=None, parent_resource_path=None, resource_type=None,
resource_name=None, api_version=None, include_response_body=False, latest_include_preview=False):
Expand Down Expand Up @@ -1697,7 +1696,6 @@ def delete_resource(cmd, resource_ids=None, resource_group_name=None,
return _single_or_collection(results)


# pylint: unused-argument
def update_resource(cmd, parameters, resource_ids=None,
resource_group_name=None, resource_provider_namespace=None,
parent_resource_path=None, resource_type=None, resource_name=None, api_version=None,
Expand All @@ -1714,7 +1712,6 @@ def update_resource(cmd, parameters, resource_ids=None,
for id_dict in parsed_ids])


# pylint: unused-argument
def tag_resource(cmd, tags, resource_ids=None, resource_group_name=None, resource_provider_namespace=None,
parent_resource_path=None, resource_type=None, resource_name=None, api_version=None,
is_incremental=None, latest_include_preview=False):
Expand All @@ -1732,7 +1729,6 @@ def tag_resource(cmd, tags, resource_ids=None, resource_group_name=None, resourc
tags, is_incremental)) for id_dict in parsed_ids])


# pylint: unused-argument
def invoke_resource_action(cmd, action, request_body=None, resource_ids=None,
resource_group_name=None, resource_provider_namespace=None,
parent_resource_path=None, resource_type=None, resource_name=None,
Expand Down