Skip to content

Add --no-warn-on-updates option#408

Open
KAMI911 wants to merge 2 commits intoHariSekhon:masterfrom
KAMI911:master
Open

Add --no-warn-on-updates option#408
KAMI911 wants to merge 2 commits intoHariSekhon:masterfrom
KAMI911:master

Conversation

@KAMI911
Copy link

@KAMI911 KAMI911 commented Sep 7, 2023

This settings will return OK instead of WARNING even when updates are available. This is not recommended from the security standpoint, but may be wanted to disable alerts while the plugin output still shows the number of available updates.

Add  "--no-warn-on-updates" option, that returns OK instead of WARNING even when updates are available. This is not recommended from the security standpoint, but may be wanted to disable alerts while the plugin output still shows the number of available updates.
+ "available. This is not recommended from the security" \
+ "standpoint, but may be wanted to disable alerts while" \
+ "the plugin output still shows the number of available" \
+ "updates.")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

continuation line under-indented for visual indent
indentation contains mixed spaces and tabs
indentation contains tabs

help="Return OK instead of WARNING even when updates are" \
+ "available. This is not recommended from the security" \
+ "standpoint, but may be wanted to disable alerts while" \
+ "the plugin output still shows the number of available" \
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

continuation line under-indented for visual indent
indentation contains mixed spaces and tabs
indentation contains tabs
the backslash is redundant between brackets

dest="no_warn_on_updates",
help="Return OK instead of WARNING even when updates are" \
+ "available. This is not recommended from the security" \
+ "standpoint, but may be wanted to disable alerts while" \
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

continuation line under-indented for visual indent
indentation contains mixed spaces and tabs
indentation contains tabs
the backslash is redundant between brackets

action="store_true",
dest="no_warn_on_updates",
help="Return OK instead of WARNING even when updates are" \
+ "available. This is not recommended from the security" \
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

continuation line under-indented for visual indent
indentation contains mixed spaces and tabs
indentation contains tabs
the backslash is redundant between brackets

parser.add_option("--no-warn-on-updates",
action="store_true",
dest="no_warn_on_updates",
help="Return OK instead of WARNING even when updates are" \
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indentation contains mixed spaces and tabs
line too long (81 > 79 characters)
the backslash is redundant between brackets


parser.add_option("--no-warn-on-updates",
action="store_true",
dest="no_warn_on_updates",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

continuation line under-indented for visual indent
indentation contains mixed spaces and tabs
indentation contains tabs

+ "yum for package management")

parser.add_option("--no-warn-on-updates",
action="store_true",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

continuation line under-indented for visual indent
indentation contains mixed spaces and tabs
indentation contains tabs

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@KAMI911
Copy link
Author

KAMI911 commented Sep 21, 2023

I'll fix these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant