fix: License check fails on Flutter packages

[mtwichel]

Description

When adding the license check workflow to a Flutter project, I received

Because square_online_ordering_app requires the Flutter SDK, version solving failed.

Flutter users should use `flutter pub` instead of `dart pub`.

I suspect this is because we're running dart pub get --no-example here instead of flutter pub get.

Steps To Reproduce

1. Set up Flutter project and GitHub actions with the verify license check.
2. Run the action.

Expected Behavior

I expect it to not fail.

Additional Context

No response

[alestiago]

Hi @mtwichel ! Thanks for opening an issue.

I haven't invested time into reproducing the issue. Assuming it is a valid issue, I see two solutions we could do on our end to resolve: (a) introduce a new workflow, perhaps named flutter_license_check or (b) introduce a new option or options, for example introducing an option named flutter_sdk where if specified, the given Flutter SDK version is installed and use the flutter tool to get packages

I haven't yet invested time into evaluating the pros and cons for both approaches.

I'll cc @wolfenrain in case he wants to drop his two cents.

[tomarra]

Talked through this in ticket grooming and have a plan forward

• Update the workflow to allow passing in a Flutter version
• If a Flutter version and Dart version is passed in, a warning should be thrown to let the user know that the Dart version will be ignored and Flutter version takes precedence.
• If a Flutter version is passed in then the "Setup" step needs to install Flutter instead of Dart
• If a Flutter version is passed in then the "Install Dependencies" step needs to use the flutter command instead of the dart command.