Skip to content

Omit obvious local variable types and reformat with a line length of 100 - #1145

Merged
JSUYA merged 2 commits into
flutter-tizen:mainfrom
JSUYA:analysis-options-alignment
Sep 16, 2026
Merged

JSUYA merged 2 commits into
flutter-tizen:mainfrom
JSUYA:analysis-options-alignment

Conversation

@JSUYA

@JSUYA JSUYA commented Sep 7, 2026

Copy link
Copy Markdown
Member

Migrate the packages governed by the root analysis_options.yaml off the old always_specify_types style with dart fix --code=omit_obvious_local_variable_types, reformat them with dart format, then enable both omit_obvious_local_variable_types and formatter: page_width: 100.

(#1089)

@JSUYA JSUYA self-assigned this Sep 7, 2026
@JSUYA JSUYA added the bug Something isn't working label Sep 7, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 7, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-07T10:19:10.223226Z e42063d PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@JSUYA
JSUYA force-pushed the analysis-options-alignment branch 2 times, most recently from c385fcc to 63c1191 Compare September 11, 2026 07:42
Migrate the packages governed by the root analysis_options.yaml off the old
always_specify_types style with
`dart fix --code=omit_obvious_local_variable_types`, reformat them with
`dart format`, then enable both `omit_obvious_local_variable_types` and
`formatter: page_width: 100`
(flutter-tizen#1089).
@JSUYA
JSUYA force-pushed the analysis-options-alignment branch from 63c1191 to e347eeb Compare September 16, 2026 08:09
@JSUYA
JSUYA merged commit e10e2bc into flutter-tizen:main Sep 16, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment