-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[location_background] Ignore for CI #1890
Conversation
amirh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this requires bumping the minimal Flutter version required to the first release where the template parameter was added.
This pacakge should be removed entirely soon, so remove it from CI so we don't spend further time on analyzer failures.
Blacklisted it from our CI checks instead like we discussed offline. |
|
CI seems to still be failing, e.g: https://cirrus-ci.com/task/5899696759898112 |
|
I'll fix the location_background issue in #1884 but this is still worth investigating |
|
@amirh @collinjackson figured out the underlying issue. I missed a branch in |
This pacakge should be removed entirely soon, so remove it from CI so we don't spend further time on analyzer failures.
This pacakge should be removed entirely soon, so remove it from CI so we don't spend further time on analyzer failures.
* Initial release of cloud_functions_web
Description
This pacakge should be removed entirely soon, so remove it from CI so we
don't spend further time on analyzer failures.
Related Issues
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]). This will ensure a smooth and quick review process.///).flutter analyze) does not report any problems on my PR.Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?