Skip to content
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/run linting and fix any errors #158

Merged
merged 8 commits into from
Jun 25, 2024

Conversation

iremozs
Copy link
Contributor

@iremozs iremozs commented Jun 23, 2024

Description

  • Run make format and make lint for both frontend and backend
  • Deleted unused variables and imports
  • Deleted unnecessary comments and logs
  • Reorganised logs and changed prints to logs
  • Reorganised imports
  • Corrected existing docstrings
  • Tested and everything seems to work fine without any problems

Related Backlog Item

Issue: #152


Context


Checklist:

  • I have documented my changes
  • I have tested the changes and they work as expected
  • I have assigned this PR to someone
  • I have run make format to format my code

Additional references

@iremozs iremozs self-assigned this Jun 23, 2024
@iremozs iremozs requested a review from nikolas-rauscher June 23, 2024 18:56
@nikolas-rauscher nikolas-rauscher merged commit 850aaf3 into develop Jun 25, 2024
1 check passed
@nikolas-rauscher nikolas-rauscher deleted the refactor/run-linting-and-fix-any-errors branch July 6, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants