• liyuqian's avatar
    Add LICENSE test to presubmit checks (#28369) · dd5559a5
    liyuqian authored
    ## Description
    
    Also update the existing dart files with missing licenses.
    
    Without the fix, we'll emit the following error message
    ```
    License headers cannot be found at the beginning of the following files.
    
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter/lib/src/animation/tween_sequence.dart
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter/test/material/raw_material_button_test.dart
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter/test/widgets/async_lifecycle_test.dart
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter/test/widgets/sliver_constraints_test.dart
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter/test/widgets/app_test.dart
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter/test/widgets/test_border.dart
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter/test/widgets/physical_model_test.dart
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter/test/widgets/inherited_model.dart
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter_tools/lib/src/base/user_messages.dart
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter_tools/test/src/pubspec_schema.dart
    /usr/local/google/home/liyuqian/flutter/flutter/packages/flutter_tools/test/ios/simulators_test.dart
    ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
    ```
    
    ## Related Issues
    
    Fixes https://github.com/flutter/flutter/issues/28368
    
    ## 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.
    
    - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
    - [x] My PR includes tests for *all* changed/updated/fixed behaviors (See [Test Coverage]).
    - [x] All existing and new tests are passing.
    - [x] I updated/added relevant documentation (doc comments with `///`).
    - [x] The analyzer (`flutter analyze --flutter-repo`) does not report any problems on my PR.
    - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
    - [x] I signed the [CLA].
    - [x] I am willing to follow-up on review comments in a timely manner.
    
    ## Breaking Change
    
    Does your PR require Flutter developers to manually update their apps to accommodate your change?
    
    - [ ] Yes, this is a breaking change (Please read [Handling breaking changes]).
    - [x] No, this is *not* a breaking change.
    
    <!-- Links -->
    [issue database]: https://github.com/flutter/flutter/issues
    [Contributor Guide]: https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
    [Test Coverage]: https://github.com/flutter/flutter/wiki/Test-coverage-for-package%3Aflutter
    [Flutter Style Guide]: https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
    [Features we expect every widget to implement]: https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#features-we-expect-every-widget-to-implement
    [CLA]: https://cla.developers.google.com/
    [Handling breaking changes]: https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
    dd5559a5
tooltip_test.dart 24.7 KB