Skip to content

FEDX-4095: Resolve dart3 formatting issues#47

Merged
btr-rmconsole-5[bot] merged 2 commits intomasterfrom
resolve_dart3_formatting_issues
Oct 17, 2025
Merged

FEDX-4095: Resolve dart3 formatting issues#47
btr-rmconsole-5[bot] merged 2 commits intomasterfrom
resolve_dart3_formatting_issues

Conversation

@matthewnitschke-wk
Copy link
Copy Markdown
Contributor

@matthewnitschke-wk matthewnitschke-wk commented Oct 17, 2025

FEDX-4095

Issue Status

There's a nuanced issue with the checks workflow in the format job

Since many of workiva repos rely on the package workiva_analysis_options in their analysis_options.yaml file, and content within that file can effect formatting results, if you dont run pub-get to resolve that dependency before running formatting, you will get fmt changes.

See Workiva/dependency_validator#160 for an example of this issue

This PR simply runs pub get before running dart format to resolve these issues

@matthewnitschke-wk matthewnitschke-wk changed the title resolve dart3 formatting issues Resolve dart3 formatting issues Oct 17, 2025
@matthewnitschke-wk matthewnitschke-wk marked this pull request as ready for review October 17, 2025 03:01
@matthewnitschke-wk matthewnitschke-wk requested a review from a team as a code owner October 17, 2025 03:01
@bender-wk bender-wk changed the title Resolve dart3 formatting issues FEDX-4095: Resolve dart3 formatting issues Oct 17, 2025
@alanknight-wk
Copy link
Copy Markdown

@Workiva/release-management-pp

Copy link
Copy Markdown

@rmconsole-wf rmconsole-wf left a comment

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-5 btr-rmconsole-5 bot merged commit 917aca0 into master Oct 17, 2025
35 of 40 checks passed
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.

3 participants