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

spread: update flutter test #4015

Merged
merged 1 commit into from
Jan 12, 2023
Merged

spread: update flutter test #4015

merged 1 commit into from
Jan 12, 2023

Conversation

sergiusens
Copy link
Collaborator

The test had sdk bounds that the latest master would not work with (new DART), while removing the bounds, the pubspec was outdated enough.

Moved to the new flutter template post linux desktop beta announcement. The downside is that <app> --help now requires X (could be solved with xfvb) but alas, does not print help and exit.

Signed-off-by: Sergio Schvezov sergio.schvezov@canonical.com

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run make lint?
  • Have you successfully run pytest tests/unit?

The test had sdk bounds that the latest master would not work with (new DART),
while removing the bounds, the pubspec was outdated enough.

Moved to the new flutter template post linux desktop beta announcement. The
downside is that `<app> --help` now requires X (could be solved with xfvb) but
alas, does not print help and exit.

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
@codecov-commenter
Copy link

Codecov Report

Merging #4015 (d73ea09) into main (0eb1942) will not change coverage.
The diff coverage is n/a.

❗ Current head d73ea09 differs from pull request most recent head ef2f072. Consider uploading reports for the commit ef2f072 to get more accurate results

@@           Coverage Diff           @@
##             main    #4015   +/-   ##
=======================================
  Coverage   91.61%   91.61%           
=======================================
  Files          77       77           
  Lines        5153     5153           
=======================================
  Hits         4721     4721           
  Misses        432      432           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sergiusens sergiusens enabled auto-merge (squash) January 12, 2023 22:37
@sergiusens sergiusens merged commit 202e912 into main Jan 12, 2023
@sergiusens sergiusens deleted the flutter-test branch January 12, 2023 22:49
cmatsuoka pushed a commit to cmatsuoka/snapcraft that referenced this pull request Feb 8, 2023
The test had sdk bounds that the latest master would not work with (new DART),
while removing the bounds, the pubspec was outdated enough.

Moved to the new flutter template post linux desktop beta announcement. The
downside is that `<app> --help` now requires X (could be solved with xfvb) but
alas, does not print help and exit.

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
cmatsuoka added a commit that referenced this pull request Feb 8, 2023
* tests: update ruby bundle v1 spread test (#4013)
* spread: update flutter test (#4015)

Co-authored-by: Callahan Kovacs <callahan.kovacs@canonical.com>
Co-authored-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
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.

3 participants