Solution #1727
Solution #1727
Conversation
There was a problem hiding this comment.
Great job on the pull request! 🎉 While there's a minor suggestion to avoid code duplication by using the format_linter_error function, it doesn't impact the functionality. Keep up the good work and consider refactoring for cleaner code in the future!
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| "line": error["line_number"], | ||
| "column": error["column_number"], | ||
| "message": error["text"], | ||
| "name": error["code"], | ||
| "source": "flake8" |
There was a problem hiding this comment.
This block of code is duplicated in the format_linter_error function. Consider using the format_linter_error function to avoid repetition and adhere to the 'Don't Repeat Yourself' principle.
app/main.py
Outdated
| { | ||
| "line": mistake["line_number"], | ||
| "column": mistake["column_number"], | ||
| "message": mistake["text"], | ||
| "name": mistake["code"], | ||
| "source": "flake8" |
There was a problem hiding this comment.
This block of code is also duplicated in the format_linter_error function. Consider using the format_linter_error function to avoid repetition and adhere to the 'Don't Repeat Yourself' principle.
No description provided.