sweepai / sweep

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

Sweep: In on_ticket.py make sure everything uses loguru and loguru.exception instead of traceback #2135

Open wwzeng1 opened 1 year ago

wwzeng1 commented 1 year ago

But do not touch the tracebacks called in log_error calls, they're still needed.

Checklist - [X] ``sweepai/handlers/on_ticket.py`` ✅ Commit [`9832c30`](https://github.com/sweepai/sweep/commit/9832c30b3194f98960b8a13597733c410ea7c105)
sweep-nightly[bot] commented 1 year ago

Here's the PR! https://github.com/sweepai/sweep/pull/2162.

💎 Sweep Pro: I'm creating this ticket using GPT-4. You have unlimited GPT-4 tickets.

Actions (click)


Step 1: 🔎 Searching

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description. https://github.com/sweepai/sweep/blob/e1bd3e4dca62441b91a874c2cf7c041f2615a891/sweepai/handlers/on_ticket.py#L1065-L1322 https://github.com/sweepai/sweep/blob/e1bd3e4dca62441b91a874c2cf7c041f2615a891/sweepai/agents/graph_child.py#L672-L728 https://github.com/sweepai/sweep/blob/e1bd3e4dca62441b91a874c2cf7c041f2615a891/sweepai/handlers/on_ticket.py#L558-L638

Step 2: ⌨️ Coding


Step 3: 🔁 Code Review

I have finished reviewing the code for completeness. I did not find errors for sweep/replace-traceback-with-loguru_8.

.


🎉 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. Join Our Discord