Fix: Do not leak codefix IDs in API#1775
Fix: Do not leak codefix IDs in API#1775Dedsec0098 wants to merge 4 commits intoaboutcode-org:mainfrom
Conversation
|
Hey @Dedsec0098, Great to see your enthusiasm for contributing! It’s awesome that you’re actively working on PRs. Just wanted to share a few tips that might help make your contributions even more impactful: Now, moving on to some specific review points for this PR… |
|
Hi @Rishi-source thanks a lot for sharing me your views, I totally get your point and would definitely start putting test results in my PRs. |
|
@Dedsec0098 please add tests, rest it looks good. Thanks! |
|
Sure @TG1999 , I'll start working on adding tests |
|
Hey @TG1999 I have added tests to my PR, please let me know if I have done it correctly or some changes are needed. |
| @@ -0,0 +1,46 @@ | |||
| import re | |||
There was a problem hiding this comment.
Tests are failing pls check!
There was a problem hiding this comment.
Hey @TG1999 I am finding it difficult to solve these tests issues, but i am trying to solve them as soon as possible.
There was a problem hiding this comment.
I have configured all the files and now all the tests are passing successfully.
Signed-off-by: Shrish0098 <shrish409@gmail.com>
Signed-off-by: Shrish0098 <shrish409@gmail.com>
Signed-off-by: Shrish0098 <shrish409@gmail.com>
5762915 to
ab7f19d
Compare
Signed-off-by: Shrish0098 <shrish409@gmail.com>
|
This is not actionable. Please take the time to learn how to code and stop submitting code that does not run and is AI-generated, wasting precious review time from the maintainers. |

Fix #1752
Added uuid instead of using id
Imported uuid in the model.py
Added test_codefix_id under tests section
Signed-off-by: Shrish Mishra shrish409@gmail.com