Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: reduce number of used scaffolds #811

Merged
merged 2 commits into from Dec 12, 2023
Merged

fix: reduce number of used scaffolds #811

merged 2 commits into from Dec 12, 2023

Conversation

Feichtmeier
Copy link
Member

@Feichtmeier Feichtmeier commented Dec 12, 2023

grafik

Ref #728

@Feichtmeier
Copy link
Member Author

Since the YMD changed, the golden tests for those widgets changed. Even though the visual look does not change but the layers of widgets.
Since my host is on 3.16 I can not properly push updated goldens
Any idea how we could fix this @Jupi007 @d-loose ? Except, me using different flutter versions :S

@Jupi007
Copy link
Member

Jupi007 commented Dec 12, 2023

What we can do is setup a github action for this 🤔

@Feichtmeier
Copy link
Member Author

We have

https://github.com/ubuntu/yaru_widgets.dart/blob/main/.github/workflows/nightly.yml

but it is flutter 3.10 xD

@Feichtmeier
Copy link
Member Author

@Jupi007
since very precisely YMD goldens do not match anymore

❌ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_master_detail_page_test.dart: golden images (variant: landscape-light: themeMode: ThemeMode.light, value: Instance of 'YaruMasterDetailGoldenVariant') (failed)
❌ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_master_detail_page_test.dart: golden images (variant: landscape-dark: themeMode: ThemeMode.dark, value: Instance of 'YaruMasterDetailGoldenVariant') (failed)
✅ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_master_detail_page_test.dart: golden images (variant: portrait-subtitle-light: themeMode: ThemeMode.light, value: Instance of 'YaruMasterDetailGoldenVariant')
✅ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_master_detail_page_test.dart: golden images (variant: portrait-subtitle-dark: themeMode: ThemeMode.dark, value: Instance of 'YaruMasterDetailGoldenVariant')
❌ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_master_detail_page_test.dart: golden images (variant: landscape-subtitle-light: themeMode: ThemeMode.light, value: Instance of 'YaruMasterDetailGoldenVariant') (failed)
❌ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_master_detail_page_test.dart: golden images (variant: landscape-subtitle-dark: themeMode: ThemeMode.dark, value: Instance of 'YaruMasterDetailGoldenVariant') (failed)
✅ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_master_detail_page_test.dart: golden images (variant: portrait-spacer-light: themeMode: ThemeMode.light, value: Instance of 'YaruMasterDetailGoldenVariant')
✅ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_master_detail_page_test.dart: golden images (variant: portrait-spacer-dark: themeMode: ThemeMode.dark, value: Instance of 'YaruMasterDetailGoldenVariant')
✅ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_linear_progress_indicator_test.dart: - YaruLinearProgressIndicator Test
✅ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_linear_progress_indicator_test.dart: golden images (variant: indeterminate-light: themeMode: ThemeMode.light, value: null)
❌ /home/runner/work/yaru_widgets.dart/yaru_widgets.dart/test/widgets/yaru_master_detail_page_test.dart: golden images (variant: landscape-spacer-light: themeMode: ThemeMode.light, value: Instance of 'YaruMasterDetailGoldenVariant') (failed)

could we merge this one and then regen them tonight with the CI? (at least I think this should work)

@Feichtmeier Feichtmeier marked this pull request as ready for review December 12, 2023 19:52
Copy link
Member

@Jupi007 Jupi007 left a comment

Choose a reason for hiding this comment

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

Great, LGTM 👍

@Feichtmeier
Copy link
Member Author

Okay let's hope this plan works :D 🤞
Otherwise we can still revert this

@Feichtmeier Feichtmeier merged commit a40ab77 into main Dec 12, 2023
6 of 7 checks passed
@Feichtmeier Feichtmeier deleted the minus_one_scaffold branch December 12, 2023 20:11
@github-actions github-actions bot mentioned this pull request Dec 12, 2023
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.

None yet

2 participants