-
Notifications
You must be signed in to change notification settings - Fork 77
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
Dev into master #9
Conversation
lucifer/Errors.py
Outdated
notifier.notify(err) | ||
print("The Following Error Occurred In Current Module, Reported To Devs...\n"+str(err)) | ||
|
||
|
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.
Blank line at end of file
lucifer/GUI/GUI.py
Outdated
"To Fix this problem please delete or edit the 'settings.yml' file!") | ||
else: | ||
Settings.create_settings(self.font) | ||
|
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.
Blank line at end of file
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.
The PR diff size of 128774 lines exceeds the maximum allowed for the inline comments feature.
Here is an overview of what got changed by this pull request: Issues
======
- Added 6
Complexity increasing per file
==============================
- lucifer/Requirements.py 8
- lucifer/Errors.py 2
- lucifer/Settings.py 4
See the complete overview on Codacy |
lucifer/Settings.py
Outdated
if settings: | ||
try: | ||
return reduce(operator.getitem, setting_split, settings) | ||
except KeyError as e: |
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.
Codacy found an issue: local variable 'e' is assigned to but never used (F841)
lucifer/Settings.py
Outdated
return reduce(operator.getitem, setting_split, settings) | ||
except KeyError as e: | ||
raise LuciferSettingNotFound(setting) | ||
except TypeError as e: |
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.
Codacy found an issue: local variable 'e' is assigned to but never used (F841)
lucifer/Settings.py
Outdated
if settings: | ||
try: | ||
return reduce(operator.getitem, setting_split, settings) | ||
except KeyError as e: |
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.
Codacy found an issue: Unused variable 'e'
@@ -0,0 +1 @@ | |||
luciferManager = None |
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.
Codacy found an issue: Missing docstring in public package (D104)
lucifer/GUI/Console.py
Outdated
tags.append("stdout") | ||
else: | ||
tags.append(mapped_tag) | ||
return *tags, |
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.
Codacy found an issue: invalid syntax (F999)
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.
The PR diff size of 128773 lines exceeds the maximum allowed for the inline comments feature.
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.
The PR diff size of 128777 lines exceeds the maximum allowed for the inline comments feature.
Code Climate has analyzed commit 53ecffd and detected 0 issues on this pull request. View more on Code Climate. |
Settings changes (yml)