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

Clean up outdated comments #1414

Open
hoffie opened this issue Mar 31, 2021 · 0 comments
Open

Clean up outdated comments #1414

hoffie opened this issue Mar 31, 2021 · 0 comments
Labels
refactoring Non-behavioural changes, Code cleanup
Milestone

Comments

@hoffie
Copy link
Member

hoffie commented Mar 31, 2021

  • Remove commented out code as it is confusing. It will still be part of the git history if needed again.
  • Check if TODO's are actionable
    • If they are, open issues for them and remove them.
    • If they are, but they are deprecation reminders, keep them.
    • If they aren't, drop them.
  • Check if old compatibility code can be dropped (AT LEAST QT 4.8 IS REQUIRED?)
  • Check "temporary workaround"-style comments. Check if the workaround can be removed or remove the comment if the workaround has proven to stay required.
@hoffie hoffie added the refactoring Non-behavioural changes, Code cleanup label Mar 31, 2021
@ann0see ann0see added this to Tracking Jul 1, 2023
@github-project-automation github-project-automation bot moved this to Triage in Tracking Jul 1, 2023
@ann0see ann0see added this to the Release 4.0.0 milestone Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Non-behavioural changes, Code cleanup
Projects
Status: Triage
Development

No branches or pull requests

2 participants