Phil Quitslund

Results 122 issues of Phil Quitslund

A meta-issue to discuss and track server work on quick assists and fixes for [patterns](https://github.com/dart-lang/language/blob/master/working/0546-patterns/patterns-feature-specification.md). **CorrectionProducers to Review/Test/Update** - [ ] `RemoveDuplicateCase` - [ ] `UseCurlyBraces`

P1
area-analyzer
analyzer-server

The following code generates what look like parser errors. ![image](https://user-images.githubusercontent.com/67586/188941239-e44ba5a9-5224-406f-9588-a210987dfec7.png) ``` error: Annotation must be either a const variable reference or const constructor invocation. (invalid_annotation at [sandbox] lib/public.dart:27) error: Variables...

records

Follow-up from https://github.com/flutter/devtools/pull/1837#discussion_r411544355, where @jacobr suggested: > nit: consider using List comprehensions for this case instead of map followed by toList(). > this might be a reasonable general lint to...

type-enhancement
lint request
P3

**Ensure that SQL query strings are always constant, or use whitelisted string-interpolation methods.** Specifically guard against unsafe use of [package:sqflite](https://github.com/tekartik/sqflite/tree/master/sqflite). _(Placeholder. Details to follow.)_

type-enhancement
lint request
customer: money (g3)
P4

**Avoid default clauses in enum-like switch statements** Good: ```dart switch (testEnum) { case TestEnum.A: return '123'; case TestEnum.B: return 'abc'; } // Default here. return null; ``` Bad: ```dart switch...

type-enhancement
lint request
customer: money (g3)
P4

To enable runtime language selection, results from calling `Intl.message` should not be stored in static final, or objects that hold them.

type-enhancement
lint request
customer: money (g3)
P4

See this fix for a place this would have been handy: https://dart-review.googlesource.com/c/sdk/+/121862 /fyi @stereotype441 @srawlins

type-enhancement
lint request

TL;DR: add lints to lint rules in analysis options that duplicate included rules (e.g., pedantic). More context in https://github.com/dart-lang/pedantic/issues/23, where @kentcb wrote: > Here's my situation: I want to use...

type-enhancement
lint request
P3

From the linter pana report: ![image](https://user-images.githubusercontent.com/67586/53369773-cd614d00-3900-11e9-93c6-6c12d8af3825.png)

type-enhancement
lint request
pub
P3

In https://github.com/flutter/flutter/pull/27791#discussion_r255772365, @goderbauer wrote: > That a dartdoc should start with a one-sentence summary as its own paragraph is something I have to frequently point out in code reviews -...

type-enhancement
lint request
customer: flutter
P2