Skip to content
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.

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

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

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

@@ -684,6 +694,15 @@ def main():
+ "intermittently pop up when someone is running " \
+ "yum for package management")

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

@@ -684,6 +694,15 @@ def main():
+ "intermittently pop up when someone is running " \
+ "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

@sonarcloud
Copy link

sonarcloud 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.

None yet

1 participant