devtools
[CQ] cleanup `unnecessary_ignore`s
#8825
Merged
Overview
Commits
2
Changes
View On GitHub

[CQ] cleanup `unnecessary_ignore`s #8825

pq merged 2 commits into flutter:master from pq:cq_unnecessary_ignores
pq
pq184 days ago (edited 184 days ago)

Adds the unnecessary_ignore lint and cleans up some unneeded ignore comments.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or there is a reason for not adding tests.

build.yaml badge

If you need help, consider asking for help on Discord.

pq [CQ] enforce and cleanup `unnecessary_ignore`s
809ddbaf
pq pq requested a review from kenzieschmoll kenzieschmoll 184 days ago
pq pq requested a review from bkonyi bkonyi 184 days ago
pq pq requested a review from elliette elliette 184 days ago
pq pq requested a review 184 days ago
pq pq added release-notes-not-required
pq
pq184 days ago

Huh. There's something funny in the analysis of the case_study apps. I'll remove the lint enforcement for now.

pq (revert)
24a46832
pq pq changed the title [CQ] enforce and cleanup `unnecessary_ignore`s [CQ] cleanup `unneccessary_ignore`s 184 days ago
pq pq changed the title [CQ] cleanup `unneccessary_ignore`s [CQ] cleanup `unnecessary_ignore`s 184 days ago
bkonyi
bkonyi approved these changes on 2025-01-30
pq pq merged 1bdcf7a7 into master 183 days ago

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
release-notes-not-required
Milestone
No milestone
Terms Of Use Privacy Policy Refund Policy Impressum