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.

Sign up for GitHub

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

Jump to bottom

Lint rules suggestion. #137

Open
Chralu opened this issue Dec 26, 2024 · 0 comments · May be fixed by #138
Open

Lint rules suggestion. #137

Chralu opened this issue Dec 26, 2024 · 0 comments · May be fixed by #138

Comments

@Chralu
Copy link
Contributor

Chralu commented Dec 26, 2024

Quality improvement Suggestion

There are a few technical issues which could be prevented by lints.
In the same way, Lints could enlighten syntax improvements.

The rules I have in mind to prevent issues :

  • unawaited futures
  • BuildContexts used across asynchronous gaps
  • switch statements missing cases

The ones to improve syntax :

  • Use named constants instead
  • Fix code references in comments
  • Throw error subclasses only
Chralu added a commit to Chralu/anytime_podcast_player that referenced this issue Dec 26, 2024
@Chralu
Lint rules suggestion.
3b73fd6 
Fixes amugofjava#137
@Chralu Chralu linked a pull request Dec 26, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Lint rules suggestion. Chralu/anytime_podcast_player
1 participant
@Chralu