Skip to content

Feature/migration to proper linter setup#1

Merged
maximsan merged 8 commits into
mainfrom
feature/migration-to-proper-linter-setup
Jun 13, 2026
Merged

Feature/migration to proper linter setup#1
maximsan merged 8 commits into
mainfrom
feature/migration-to-proper-linter-setup

Conversation

@maximsan

Copy link
Copy Markdown
Owner

No description provided.

maximsan added 8 commits June 9, 2026 12:53
…tion

Replace flutter_lints with very_good_analysis and add dart_code_linter
(analyzer-12 compatible, loaded via the analysis_options plugins block
alongside riverpod_lint). Enable no-magic-number + per-function/class
size/complexity metrics, with CustomPainter and theme-token exclusions.
Keep directives_ordering, drop cascade_invocations; scope
public_member_api_docs to real APIs (token/DTO/infra files excluded via
ignore_for_file).

Begin remediation: core widgets and onboarding welcome documented, magic
numbers extracted to named constants.
… numbers extracted to named constants part 3
…gate CI

Finish no-magic-number remediation (learn/game-type/premium/settings + token
mappings to AppSpacing); raise source-lines-of-code metric to 90 for declarative
build methods; clear very_good full-tree analyze (disable sort_pub_dependencies,
wrap long test lines); add dart_code_linter metrics step to CI; update
CLAUDE/AGENTS/README to reflect lint-enforced magic numbers + per-function size.
… widgets

Decompose learn / lesson-completion / game-type-practice / module-detail /
path-node files (271–437 lines) into one-public-widget-per-file units; extract
LessonCompletionBody as a pure (DB-free) view + reward DTO.
@maximsan
maximsan force-pushed the feature/migration-to-proper-linter-setup branch from e31c1c8 to c850a0d Compare June 13, 2026 18:22
@maximsan
maximsan merged commit f3f52ef into main Jun 13, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant