sweepai / sweep

Sweep: open-source AI-powered Software Developer for small features and bug fixes.
https://sweep.dev
Other
7.43k stars 425 forks source link

Sweep: Refactor test_jedi_code.py to remove commented code and add traceback in exception handling #1877

Open sweep-nightly[bot] opened 1 year ago

sweep-nightly[bot] commented 1 year ago

In the file tests/archive/test_jedi_code.py, there are two main issues that need to be addressed:

  1. Large chunks of code are commented out. This is against our coding standards as it makes the code harder to read and understand. The commented code starts from the line # if __name__ == "__main__": and continues till the end of the file. Please remove these commented lines or refactor them into smaller, reusable functions if they are still needed.

  2. There are several instances where exceptions are caught but not logged using traceback. This makes it harder to debug issues when they arise. For example, in the get_full_inference_name_and_type method, an exception is caught but not logged. Please add traceback logging to all exceptions to improve our error handling.

Please make these changes and submit a new pull request for review.

sweep-nightly[bot] commented 1 year ago
Sweeping

0%

Actions (click)


❌ Unable to Complete PR

It looks like an issue has occurred around fetching the files. Perhaps the repo has not been initialized. If this error persists contact team@sweep.dev.

@sweep-nightly[bot], editing this issue description to include more details will automatically make me relaunch.

For bonus GPT-4 tickets, please report this bug on Discord.


🎉 Latest improvements to Sweep:


💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request.