-
Notifications
You must be signed in to change notification settings - Fork 676
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
Fix Scrutinizer ssg module issues - #1 #2975
Fix Scrutinizer ssg module issues - #1 #2975
Conversation
Hello @cipherboy! Thanks for updating the PR. Cheers ! There are no PEP8 issues in this Pull Request. 🍻 Comment last updated on June 15, 2018 at 17:42 Hours UTC |
ssg/_oval.py
Outdated
defname = _add_elements(body, header) | ||
if defname == None: | ||
print("Error while evaluating oval: defname not set; missing " | ||
"definitions section?") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
print this to stderr via file=sys.stderr
1c01a2a
to
0eb9388
Compare
0eb9388
to
a565756
Compare
Signed-off-by: Alexander Scheel <ascheel@redhat.com>
Signed-off-by: Alexander Scheel <ascheel@redhat.com>
Signed-off-by: Alexander Scheel <ascheel@redhat.com>
Signed-off-by: Alexander Scheel <ascheel@redhat.com>
Signed-off-by: Alexander Scheel <ascheel@redhat.com>
Signed-off-by: Alexander Scheel <ascheel@redhat.com>
Signed-off-by: Alexander Scheel <ascheel@redhat.com>
Signed-off-by: Alexander Scheel <ascheel@redhat.com>
a565756
to
1d2a0e6
Compare
The inspection completed: 38 new issues, 2 updated code elements |
Of course they don't show how many issues I fixed. :P
@mpreisler This is ready for a review / merge. |
ACK |
Description:
First of PRs to fix various issues that Scrutinizer found during and before the move. All should be relatively minor issues. Also will fix feedback from PEP8 in most cases.