update example and some android gradle updates

.vscode/launch.json

@@ -0,0 +1,14 @@
+{
+    // Use IntelliSense to learn about possible attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "name": "Flutter",
+            "request": "launch",
+            "type": "dart",
+            "program": "example/main.dart"
+        }
+    ]
+}

analysis_options.yaml

@@ -20,133 +20,132 @@
 # in sync with this file.
 
 analyzer:
-  language:
-    enableSuperMixins: true
-  strong-mode:
-    implicit-dynamic: false
-  errors:
-    # treat missing required parameters as a warning (not a hint)
-    missing_required_param: warning
-    # treat missing returns as a warning (not a hint)
-    missing_return: warning
-    # allow having TODOs in the code
-    todo: ignore
-  exclude:
-    # for Travis - don't try to analyze the flutter repo
-    - 'flutter/**'
-
+    # language:
+    #   enableSuperMixins: true//deprecated
+    strong-mode:
+        implicit-dynamic: false
+    errors:
+        # treat missing required parameters as a warning (not a hint)
+        missing_required_param: warning
+        # treat missing returns as a warning (not a hint)
+        missing_return: warning
+        # allow having TODOs in the code
+        todo: ignore
+    exclude:
+        # for Travis - don't try to analyze the flutter repo
+        - "flutter/**"
 
 linter:
-  rules:
-    # these rules are documented on and in the same order as
-    # the Dart Lint rules page to make maintenance easier
-    # https://github.com/dart-lang/linter/blob/master/example/all.yaml
-    - always_declare_return_types
-    - always_put_control_body_on_new_line
-    # - always_put_required_named_parameters_first # we prefer having parameters in the same order as fields https://github.com/flutter/flutter/issues/10219
-    - always_require_non_null_named_parameters
-    - always_specify_types
-    - annotate_overrides
-    # - avoid_annotating_with_dynamic # conflicts with always_specify_types
-    - avoid_as
-    # - avoid_bool_literals_in_conditional_expressions # not yet tested
-    # - avoid_catches_without_on_clauses # we do this commonly
-    # - avoid_catching_errors # we do this commonly
-    - avoid_classes_with_only_static_members
-    - avoid_empty_else
-    - avoid_function_literals_in_foreach_calls
-    - avoid_init_to_null
-    - avoid_null_checks_in_equality_operators
-    # - avoid_positional_boolean_parameters # not yet tested
-    # - avoid_private_typedef_functions # we prefer having typedef (discussion in https://github.com/flutter/flutter/pull/16356)
-    - avoid_relative_lib_imports
-    - avoid_renaming_method_parameters
-    - avoid_return_types_on_setters
-    # - avoid_returning_null # we do this commonly
-    # - avoid_returning_this # https://github.com/dart-lang/linter/issues/842
-    # - avoid_setters_without_getters # not yet tested
-    # - avoid_single_cascade_in_expression_statements # not yet tested
-    - avoid_slow_async_io
-    # - avoid_types_as_parameter_names # https://github.com/dart-lang/linter/pull/954/files
-    # - avoid_types_on_closure_parameters # conflicts with always_specify_types
-    # - avoid_unused_constructor_parameters # https://github.com/dart-lang/linter/pull/847
-    - await_only_futures
-    - camel_case_types
-    - cancel_subscriptions
-    # - cascade_invocations # not yet tested
-    # - close_sinks # https://github.com/flutter/flutter/issues/5789
-    # - comment_references # blocked on https://github.com/dart-lang/dartdoc/issues/1153
-    # - constant_identifier_names # https://github.com/dart-lang/linter/issues/204
-    - control_flow_in_finally
-    - directives_ordering
-    - empty_catches
-    - empty_constructor_bodies
-    - empty_statements
-    - hash_and_equals
-    - implementation_imports
-    # - invariant_booleans # https://github.com/flutter/flutter/issues/5790
-    - iterable_contains_unrelated_type
-    # - join_return_with_assignment # not yet tested
-    - library_names
-    - library_prefixes
-    - list_remove_unrelated_type
-    # - literal_only_boolean_expressions # https://github.com/flutter/flutter/issues/5791
-    - no_adjacent_strings_in_list
-    - no_duplicate_case_values
-    - non_constant_identifier_names
-    # - omit_local_variable_types # opposite of always_specify_types
-    # - one_member_abstracts # too many false positives
-    # - only_throw_errors # https://github.com/flutter/flutter/issues/5792
-    - overridden_fields
-    - package_api_docs
-    - package_names
-    - package_prefixed_library_names
-    # - parameter_assignments # we do this commonly
-    - prefer_adjacent_string_concatenation
-    - prefer_asserts_in_initializer_lists
-    - prefer_bool_in_asserts
-    - prefer_collection_literals
-    - prefer_conditional_assignment
-    - prefer_const_constructors
-    - prefer_const_constructors_in_immutables
-    - prefer_const_declarations
-    - prefer_const_literals_to_create_immutables
-    # - prefer_constructors_over_static_methods # not yet tested
-    - prefer_contains
-    # - prefer_equal_for_default_values # not yet tested
-    # - prefer_expression_function_bodies # conflicts with https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#consider-using--for-short-functions-and-methods
-    - prefer_final_fields
-    - prefer_final_locals
-    - prefer_foreach
-    # - prefer_function_declarations_over_variables # not yet tested
-    - prefer_initializing_formals
-    # - prefer_interpolation_to_compose_strings # not yet tested
-    - prefer_is_empty
-    - prefer_is_not_empty
-    - prefer_single_quotes
-    - prefer_typing_uninitialized_variables
-    - recursive_getters
-    - slash_for_doc_comments
-    - sort_constructors_first
-    - sort_unnamed_constructors_first
-    - super_goes_last
-    - test_types_in_equals
-    - throw_in_finally
-    # - type_annotate_public_apis # subset of always_specify_types
-    - type_init_formals
-    # - unawaited_futures # https://github.com/flutter/flutter/issues/5793
-    - unnecessary_brace_in_string_interps
-    - unnecessary_getters_setters
-    # - unnecessary_lambdas # https://github.com/dart-lang/linter/issues/498
-    - unnecessary_null_aware_assignments
-    - unnecessary_null_in_if_null_operators
-    - unnecessary_overrides
-    - unnecessary_parenthesis
-    # - unnecessary_statements # not yet tested
-    - unnecessary_this
-    - unrelated_type_equality_checks
-    - use_rethrow_when_possible
-    # - use_setters_to_change_properties # not yet tested
-    # - use_string_buffers # https://github.com/dart-lang/linter/pull/664
-    # - use_to_and_as_if_applicable # has false positives, so we prefer to catch this by code-review
-    - valid_regexps
+    rules:
+        # these rules are documented on and in the same order as
+        # the Dart Lint rules page to make maintenance easier
+        # https://github.com/dart-lang/linter/blob/master/example/all.yaml
+        - always_declare_return_types
+        - always_put_control_body_on_new_line
+        # - always_put_required_named_parameters_first # we prefer having parameters in the same order as fields https://github.com/flutter/flutter/issues/10219
+        - always_require_non_null_named_parameters
+        - always_specify_types
+        - annotate_overrides
+        # - avoid_annotating_with_dynamic # conflicts with always_specify_types
+        - avoid_as
+        # - avoid_bool_literals_in_conditional_expressions # not yet tested
+        # - avoid_catches_without_on_clauses # we do this commonly
+        # - avoid_catching_errors # we do this commonly
+        - avoid_classes_with_only_static_members
+        - avoid_empty_else
+        - avoid_function_literals_in_foreach_calls
+        - avoid_init_to_null
+        - avoid_null_checks_in_equality_operators
+        # - avoid_positional_boolean_parameters # not yet tested
+        # - avoid_private_typedef_functions # we prefer having typedef (discussion in https://github.com/flutter/flutter/pull/16356)
+        - avoid_relative_lib_imports
+        - avoid_renaming_method_parameters
+        - avoid_return_types_on_setters
+        # - avoid_returning_null # we do this commonly
+        # - avoid_returning_this # https://github.com/dart-lang/linter/issues/842
+        # - avoid_setters_without_getters # not yet tested
+        # - avoid_single_cascade_in_expression_statements # not yet tested
+        - avoid_slow_async_io
+        # - avoid_types_as_parameter_names # https://github.com/dart-lang/linter/pull/954/files
+        # - avoid_types_on_closure_parameters # conflicts with always_specify_types
+        # - avoid_unused_constructor_parameters # https://github.com/dart-lang/linter/pull/847
+        - await_only_futures
+        - camel_case_types
+        - cancel_subscriptions
+        # - cascade_invocations # not yet tested
+        # - close_sinks # https://github.com/flutter/flutter/issues/5789
+        # - comment_references # blocked on https://github.com/dart-lang/dartdoc/issues/1153
+        # - constant_identifier_names # https://github.com/dart-lang/linter/issues/204
+        - control_flow_in_finally
+        - directives_ordering
+        - empty_catches
+        - empty_constructor_bodies
+        - empty_statements
+        - hash_and_equals
+        - implementation_imports
+        # - invariant_booleans # https://github.com/flutter/flutter/issues/5790
+        - iterable_contains_unrelated_type
+        # - join_return_with_assignment # not yet tested
+        - library_names
+        - library_prefixes
+        - list_remove_unrelated_type
+        # - literal_only_boolean_expressions # https://github.com/flutter/flutter/issues/5791
+        - no_adjacent_strings_in_list
+        - no_duplicate_case_values
+        - non_constant_identifier_names
+        # - omit_local_variable_types # opposite of always_specify_types
+        # - one_member_abstracts # too many false positives
+        # - only_throw_errors # https://github.com/flutter/flutter/issues/5792
+        - overridden_fields
+        - package_api_docs
+        - package_names
+        - package_prefixed_library_names
+        # - parameter_assignments # we do this commonly
+        - prefer_adjacent_string_concatenation
+        - prefer_asserts_in_initializer_lists
+        - prefer_bool_in_asserts
+        - prefer_collection_literals
+        - prefer_conditional_assignment
+        - prefer_const_constructors
+        - prefer_const_constructors_in_immutables
+        - prefer_const_declarations
+        - prefer_const_literals_to_create_immutables
+        # - prefer_constructors_over_static_methods # not yet tested
+        - prefer_contains
+        # - prefer_equal_for_default_values # not yet tested
+        # - prefer_expression_function_bodies # conflicts with https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#consider-using--for-short-functions-and-methods
+        - prefer_final_fields
+        - prefer_final_locals
+        - prefer_foreach
+        # - prefer_function_declarations_over_variables # not yet tested
+        - prefer_initializing_formals
+        # - prefer_interpolation_to_compose_strings # not yet tested
+        - prefer_is_empty
+        - prefer_is_not_empty
+        - prefer_single_quotes
+        - prefer_typing_uninitialized_variables
+        - recursive_getters
+        - slash_for_doc_comments
+        - sort_constructors_first
+        - sort_unnamed_constructors_first
+        - super_goes_last
+        - test_types_in_equals
+        - throw_in_finally
+        # - type_annotate_public_apis # subset of always_specify_types
+        - type_init_formals
+        # - unawaited_futures # https://github.com/flutter/flutter/issues/5793
+        - unnecessary_brace_in_string_interps
+        - unnecessary_getters_setters
+        # - unnecessary_lambdas # https://github.com/dart-lang/linter/issues/498
+        - unnecessary_null_aware_assignments
+        - unnecessary_null_in_if_null_operators
+        - unnecessary_overrides
+        - unnecessary_parenthesis
+        # - unnecessary_statements # not yet tested
+        - unnecessary_this
+        - unrelated_type_equality_checks
+        - use_rethrow_when_possible
+        # - use_setters_to_change_properties # not yet tested
+        # - use_string_buffers # https://github.com/dart-lang/linter/pull/664
+        # - use_to_and_as_if_applicable # has false positives, so we prefer to catch this by code-review
+        - valid_regexps

android/.settings/org.eclipse.buildship.core.prefs

@@ -0,0 +1,2 @@
+connection.project.dir=
+eclipse.preferences.version=1

android/app/build.gradle

@@ -1,8 +1,8 @@
 def localProperties = new Properties()
 def localPropertiesFile = rootProject.file('local.properties')
 if (localPropertiesFile.exists()) {
-    localPropertiesFile.withInputStream { stream ->
-        localProperties.load(stream)
+    localPropertiesFile.withReader('UTF-8') { reader ->
+        localProperties.load(reader)
     }
 }
 
@@ -11,12 +11,21 @@ if (flutterRoot == null) {
     throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
 }
 
+def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
+if (flutterVersionCode == null) {
+    flutterVersionCode = '1'
+}
+
+def flutterVersionName = localProperties.getProperty('flutter.versionName')
+if (flutterVersionName == null) {
+    flutterVersionName = '1.0'
+}
+
 apply plugin: 'com.android.application'
 apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"
 
 android {
-    compileSdkVersion 25
-    buildToolsVersion '25.0.3'
+    compileSdkVersion 27
 
     lintOptions {
         disable 'InvalidPackage'
@@ -26,9 +35,9 @@ android {
         // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
         applicationId "com.yourcompany.lottieflutter"
         minSdkVersion 16
-        targetSdkVersion 25
-        versionCode 1
-        versionName "1.0"
+        targetSdkVersion 27
+        versionCode flutterVersionCode.toInteger()
+        versionName flutterVersionName
         testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
     }
 
@@ -46,7 +55,7 @@ flutter {
 }
 
 dependencies {
-    androidTestCompile 'com.android.support:support-annotations:25.4.0'
-    androidTestCompile 'com.android.support.test:runner:0.5'
-    androidTestCompile 'com.android.support.test:rules:0.5'
+    testImplementation 'junit:junit:4.12'
+    androidTestImplementation 'com.android.support.test:runner:1.0.2'
+    androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.0.2'
 }

android/app/src/main/AndroidManifest.xml

@@ -13,7 +13,6 @@
          additional functionality it is fine to subclass or reimplement
          FlutterApplication and put your custom class here. -->
     <application
-        android:debuggable="true"
         android:name="io.flutter.app.FlutterApplication"
         android:label="lottie_flutter"
         android:icon="@mipmap/ic_launcher">

android/build.gradle

@@ -1,13 +1,11 @@
 buildscript {
     repositories {
+        google()
         jcenter()
-        maven {
-            url "https://maven.google.com"
-        }
     }
 
     dependencies {
-        classpath 'com.android.tools.build:gradle:2.3.3'
+        classpath 'com.android.tools.build:gradle:3.2.1'
     }
 }
 

android/gradle/wrapper/gradle-wrapper.properties

@@ -3,4 +3,4 @@ distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-3.3-all.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-4.10.2-all.zip