Skip to content

FEDX-6626 : Run dart format and re-enable format check in CI - #337

Merged
btr-rmconsole-1[bot] merged 2 commits into
masterfrom
FEDX-6626
Aug 14, 2026
Merged

FEDX-6626 : Run dart format and re-enable format check in CI#337
btr-rmconsole-1[bot] merged 2 commits into
masterfrom
FEDX-6626

Conversation

@hemantkumar-wk

@hemantkumar-wk hemantkumar-wk commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Motivation

Follow-up to #332 (Dart 3 upgrade). The format check was disabled during the upgrade to keep the changes minimal. Now that the upgrade is merged, running dart run dart_dev format and re-enabling the format check.

Changes

  • Run dart run dart_dev format across the codebase (111 files changed)
  • Remove format-check: false from CI workflow

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@hemantkumar-wk
hemantkumar-wk marked this pull request as draft August 7, 2026 09:10
@hemantkumar-wk
hemantkumar-wk marked this pull request as ready for review August 7, 2026 11:34
@hemantkumar-wk hemantkumar-wk changed the title Run dart format and re-enable format check in CI FEDX-6626 : Run dart format and re-enable format check in CI Aug 7, 2026
@nicbiggs-wk

Copy link
Copy Markdown
Contributor

@Workiva/release-management-pp

@nicbiggs-wk

Copy link
Copy Markdown
Contributor

@Workiva/release-management-pp

@nicbiggs-wk

Copy link
Copy Markdown
Contributor

QA+1 formatting made no changes to functionality.

@rmconsole-wf rmconsole-wf left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 from RM

@btr-rmconsole-1
btr-rmconsole-1 Bot merged commit 2c4abc3 into master Aug 14, 2026
21 checks passed
@btr-rmconsole-1
btr-rmconsole-1 Bot deleted the FEDX-6626 branch August 14, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants