-
Notifications
You must be signed in to change notification settings - Fork 48
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
refactor: Clean up imports and remove unnecessary blank lines #90
Conversation
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.
Summary by GPT-4
The code changes reorder the import statements in multiple files. The changes do not affect the functionality of the code.
Suggestions
Here are some suggestions to improve the changes in this PR:
-
Maintain consistency in import order. Group imports by standard library, third-party libraries, and local modules.
-
Remove unnecessary blank lines.
Here's the updated code with these suggestions applied:
diff --git a/src/gpt/__main__.py b/src/gpt/__main__.py
index 8fcb708a..bd149a6a 100644
--- a/src/gpt/__main__.py
+++ b/src/gpt/__main__.py
@@ -3,7 +3,6 @@
from gpt_review._gpt_cli import cli
-
if __name__ == "__main__":
exit_code = cli()
sys.exit(exit_code)
diff --git a/src/gpt_review/__main__.py b/src/gpt_review/__main__.py
index 8fcb708a..bd149a6a 100644
--- a/src/gpt_review/__main__.py
+++ b/src/gpt_review/__main__.py
@@ -3,7 +3,6 @@
from gpt_review._gpt_cli import cli
-
if __name__ == "__main__":
exit_code = cli()
sys.exit(exit_code)
diff --git a/src/gpt_review/_ask.py b/src/gpt_review/_ask.py
index 89271ded..3753a801 100644
--- a/src/gpt_review/_ask.py
+++ b/src/gpt_review/_ask.py
@@ -7,11 +7,10 @@
from knack.commands import CommandGroup
from knack.util import CLIError
+import gpt_review.constants as C
from gpt_review._command import GPTCommandGroup
from gpt_review._llama_index import _query_index
from g-
Codecov Report
@@ Coverage Diff @@
## main #90 +/- ##
===========================================
- Coverage 100.00% 92.42% -7.58%
===========================================
Files 13 13
Lines 462 462
Branches 67 44 -23
===========================================
- Hits 462 427 -35
- Misses 0 31 +31
- Partials 0 4 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Description
Testing
Additional context