search

atsign-foundation / at_widgets

Flutter widgets which aid in building applications using Atsign's technology
https://docs.atsign.com
BSD 3-Clause "New" or "Revised" License
52 stars 33 forks source link

ci: Add Flutter beta channel to static analysis matrix #846

Closed cpswan closed 1 month ago

cpswan commented 1 month ago

Fixes #845

- What I did

Added another dimension to test matrix that uses beta channel in addition to stable.

- How I did it

Based on tests for at_server

- How to verify it

CI should now have double the runs.

- Description for the changelog

ci: Add Flutter beta channel to static analysis matrix

cpswan commented 1 month ago

@gkc what do you think of putting a conditional in place for the beta channel so that we don't suppress info output from flutter analyze? That way I think we'll be pushed to be more proactive in dealing with deprecation warnings before they become problematic?

gkc commented 1 month ago

@gkc what do you think of putting a conditional in place for the beta channel so that we don't suppress info output from flutter analyze? That way I think we'll be pushed to be more proactive in dealing with deprecation warnings before they become problematic?

That sounds like a very good idea

cpswan commented 1 month ago

@gkc that change is in, so ready for another review.

Lots of work to be done in #847