fastlane-plugin-flutter 0.7.1 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9c7e965f5f507ca0c826e6bcaeaad6ce8aa2530285029560b602012e9620e62f
-  data.tar.gz: 1ea5223f9669e03d80947d92fe4fde83e347c78f82949a096faf4128c7a443a3
+  metadata.gz: 2a8c5c397684224bab9705b5f8e2980ea3c49704c25b00a189dbd9fe7ae80175
+  data.tar.gz: 7e133c3e06a7a9771eebaa86125f26d125c0e20e0dd3b8312133664d396e707c
 SHA512:
-  metadata.gz: 9be42ec972dca0347f3c2d9b7fccd645ca0a505a86ab254d2f78be1a3262bb95660f61e789bf3d6cce8f50e706101564607d8a8d58236920e7be55f8dd50c732
-  data.tar.gz: 42d2827dc366ff98ed910b3e6421e8cf7c40c156a36826e583c616bc0d453c1370343308ce039fffc5c5063d8bba7e7dfa4d30bac184d9bcc464ddee4472e6c2
+  metadata.gz: 188f0c619b42b371c8c166d48e709c44c6d6f0d7943df3f4f551f024837284d6d7d7a17317c085b4f42018ee6e0e1d7aaf5cc6189cf1024f1eff23ba83d6dbc0
+  data.tar.gz: 65b3c45bb076916775e34e8d6ba34a05477eac7997aaf31a235d01173e79bc69c50b1c759124169818fb082fa890bc9ef79c522ec9fcd56b1c31aa7e3d9071a9

data/lib/fastlane/plugin/flutter/actions/flutter_action.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'fastlane/action'
 require_relative '../base/flutter_action_base'
 require_relative '../helper/flutter_helper'
@@ -9,7 +11,7 @@ module Fastlane
 
       def self.run(params)
         if params[:capture_stdout]
-          Helper::FlutterHelper.flutter(*params[:args]) do |status, output|
+          Helper::FlutterHelper.flutter(*params[:args]) do |_status, output|
             output
           end
         else
@@ -27,7 +29,7 @@ module Fastlane
             key: :args,
             env_name: 'FL_FLUTTER_ARGS',
             description: 'Arguments to Flutter command',
-            type: Array,
+            type: Array
           ),
           FastlaneCore::ConfigItem.new(
             key: :capture_stdout,
@@ -35,8 +37,8 @@ module Fastlane
             description: 'Do not print stdout of the command, but return it',
             optional: true,
             type: Boolean,
-            default_value: false,
-          ),
+            default_value: false
+          )
         ]
       end
     end

data/lib/fastlane/plugin/flutter/actions/flutter_bootstrap_action.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'fastlane/action'
 require_relative '../base/flutter_action_base'
 require_relative '../helper/flutter_helper'
@@ -14,7 +16,7 @@ module Fastlane
         if params[:android_licenses]
           Helper::FlutterBootstrapHelper.accept_licenses(
             File.join(android_sdk_root!, 'licenses'),
-            params[:android_licenses],
+            params[:android_licenses]
           )
         end
 
@@ -38,7 +40,7 @@ module Fastlane
             '--quiet',
             '--',
             FLUTTER_REMOTE_REPOSITORY,
-            flutter_sdk_root,
+            flutter_sdk_root
           )
         end
 
@@ -52,19 +54,19 @@ module Fastlane
 
         remote_hash = Helper::FlutterHelper.git(
           'ls-remote', FLUTTER_REMOTE_REPOSITORY, flutter_channel
-        ) do |status, output, errors_thread|
+        ) do |status, output, _errors_thread|
           output.split[0].strip if status.success?
         end
         local_hash = Helper::FlutterHelper.git(
           '-C', flutter_sdk_root, 'rev-parse', 'HEAD'
-        ) do |status, output, errors_thread|
+        ) do |status, output, _errors_thread|
           output.strip if status.success?
         end
 
         if !local_hash.nil? && local_hash == remote_hash
-          UI.message("Local and remote Flutter repository hashes match " \
+          UI.message('Local and remote Flutter repository hashes match ' \
                      "(#{local_hash}), no upgrade necessary. Keeping Git " \
-                     "index untouched!")
+                     'index untouched!')
           false
         else
           UI.message("Local hash (#{local_hash}) of Flutter repository " \
@@ -94,7 +96,7 @@ module Fastlane
             env_name: 'FL_FLUTTER_CHANNEL',
             description: 'Flutter SDK channel (keep existing if unset)',
             optional: true,
-            type: String,
+            type: String
           ),
           FastlaneCore::ConfigItem.new(
             key: :flutter_auto_upgrade,
@@ -102,7 +104,7 @@ module Fastlane
             description: 'Automatically upgrade Flutter when already installed',
             default_value: true,
             optional: true,
-            is_string: false, # official replacement for "type: Boolean"
+            is_string: false # official replacement for "type: Boolean"
           ),
           FastlaneCore::ConfigItem.new(
             key: :android_licenses,
@@ -111,8 +113,8 @@ module Fastlane
             '$ANDROID_SDK_ROOT/licenses/ on developer workstations. Gradle ' \
             'will refuse to install SDK unless licenses are accepted',
             optional: true,
-            type: Hash,
-          ),
+            type: Hash
+          )
         ]
       end
     end

data/lib/fastlane/plugin/flutter/actions/flutter_build_action.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'fastlane/action'
 require_relative '../base/flutter_action_base'
 require_relative '../helper/flutter_helper'
@@ -13,8 +15,8 @@ module Fastlane
 
       FASTLANE_PLATFORM_TO_BUILD = {
         ios: 'ios',
-        android: 'apk',
-      }
+        android: 'apk'
+      }.freeze
 
       def self.run(params)
         # "flutter build" args list.
@@ -22,28 +24,24 @@ module Fastlane
 
         if params[:build]
           build_args.push(params[:build])
-        else
-          if fastlane_platform = (lane_context[SharedValues::PLATFORM_NAME] ||
+        elsif (fastlane_platform = lane_context[SharedValues::PLATFORM_NAME] ||
                                   lane_context[SharedValues::DEFAULT_PLATFORM])
-            build_args.push(FASTLANE_PLATFORM_TO_BUILD[fastlane_platform])
-          else
-            UI.user_error!('flutter_build action "build" parameter is not ' \
-            'specified and cannot be inferred from Fastlane context.')
-          end
+          build_args.push(FASTLANE_PLATFORM_TO_BUILD[fastlane_platform])
+        else
+          UI.user_error!('flutter_build action "build" parameter is not ' \
+          'specified and cannot be inferred from Fastlane context.')
         end
 
         process_deprecated_params(params, build_args)
 
-        if params[:debug]
-          build_args.push('--debug')
-        end
+        build_args.push('--debug') if params[:debug]
 
-        if build_number = (params[:build_number] ||
+        if (build_number = params[:build_number] ||
                            lane_context[SharedValues::BUILD_NUMBER])
           build_args.push('--build-number', build_number.to_s)
         end
 
-        if build_name = (params[:build_name] ||
+        if (build_name = params[:build_name] ||
                          lane_context[SharedValues::VERSION_NUMBER])
           build_args.push('--build-name', build_name.to_s)
         end
@@ -84,41 +82,41 @@ module Fastlane
         ENV['GYM_WORKSPACE'] ||= 'ios/Runner.xcworkspace'
         ENV['GYM_BUILD_PATH'] ||= 'build/ios'
         ENV['GYM_OUTPUT_DIRECTORY'] ||= 'build'
-        unless ENV.include?('GYM_SCHEME')
-          # Do some parsing on args. Is there a less ugly way?
-          build_args.each.with_index do |arg, index|
-            if arg.start_with?('--flavor', '-flavor')
-              if arg.include?('=')
-                ENV['GYM_SCHEME'] = arg.split('=', 2).last
-              else
-                ENV['GYM_SCHEME'] = build_args[index + 1]
-              end
-            end
+        return if ENV.include?('GYM_SCHEME')
+
+        # Do some parsing on args. Is there a less ugly way?
+        build_args.each.with_index do |arg, index|
+          if arg.start_with?('--flavor', '-flavor')
+            ENV['GYM_SCHEME'] = if arg.include?('=')
+                                  arg.split('=', 2).last
+                                else
+                                  build_args[index + 1]
+                                end
           end
         end
       end
 
       def self.process_deprecated_params(params, build_args)
-        unless params[:codesign].nil?
-          UI.deprecated(<<-"MESSAGE")
-flutter_build parameter "codesign" is deprecated. Use
+        return if params[:codesign].nil?
 
-  flutter_build(
-    build_args: ["--#{params[:codesign] == false ? 'no-' : ''}codesign"]
-  )
+        UI.deprecated(<<~"MESSAGE")
+          flutter_build parameter "codesign" is deprecated. Use
 
-form instead.
-          MESSAGE
+            flutter_build(
+              build_args: ["--#{params[:codesign] == false ? 'no-' : ''}codesign"]
+            )
 
-          if params[:codesign] == false
-            build_args.push('--no-codesign')
-          end
-        end
+          form instead.
+        MESSAGE
+
+        return unless params[:codesign] == false
+
+        build_args.push('--no-codesign')
       end
 
       def self.process_build_output(output, build_args)
-        artifacts = output.scan(/Built (.*?)(:? \([^)]*\))?\.$/).
-                    map { |path| File.absolute_path(path[0]) }
+        artifacts = output.scan(/Built (.*?)(:? \([^)]*\))?[.]?$/)
+                          .map { |path| File.absolute_path(path[0]) }
         if artifacts.size == 1
           lane_context[SharedValues::FLUTTER_OUTPUT] = artifacts.first
         elsif artifacts.size > 1
@@ -150,7 +148,7 @@ form instead.
             env_name: 'FL_FLUTTER_BUILD',
             description: 'Type of Flutter build (e.g. apk, appbundle, ios)',
             optional: true,
-            type: String,
+            type: String
           ),
           FastlaneCore::ConfigItem.new(
             key: :debug,
@@ -158,7 +156,7 @@ form instead.
             description: 'Build a Debug version of the app if true',
             optional: true,
             type: Boolean,
-            default_value: false,
+            default_value: false
           ),
           FastlaneCore::ConfigItem.new(
             key: :codesign,
@@ -166,14 +164,14 @@ form instead.
             description: 'Set to false to skip iOS app signing. This may be ' \
             'useful e.g. on CI or when signed later by Fastlane "sigh"',
             optional: true,
-            type: Boolean,
+            type: Boolean
           ),
           FastlaneCore::ConfigItem.new(
             key: :build_number,
             env_name: 'FL_FLUTTER_BUILD_NUMBER',
             description: 'Override build number specified in pubspec.yaml',
             optional: true,
-            type: Integer,
+            type: Integer
           ),
           FastlaneCore::ConfigItem.new(
             key: :build_name,
@@ -183,14 +181,14 @@ form instead.
               NOTE: for App Store, build name must be in the format of at most 3
                     integeres separated by a dot (".").
             DESCRIPTION
-            optional: true,
+            optional: true
           ),
           FastlaneCore::ConfigItem.new(
             key: :build_args,
             description: 'An array of extra arguments for "flutter build"',
             optional: true,
-            type: Array,
-          ),
+            type: Array
+          )
         ]
       end
     end

data/lib/fastlane/plugin/flutter/actions/flutter_generate_action.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'fastlane/action'
 require_relative '../base/flutter_action_base'
 require_relative '../helper/flutter_helper'
@@ -17,7 +19,7 @@ module Fastlane
       ALL_IMPORTS_TEST_FILE = 'test/all_imports_for_coverage_test.dart'
 
       def self.run(params)
-        Helper::FlutterHelper.flutter(*%w(packages get)) {}
+        Helper::FlutterHelper.flutter(*%w[packages get]) {}
 
         if params[:coverage_all_imports] && File.exist?(ALL_IMPORTS_TEST_FILE)
           # This file may somehow confuse codegeneration (e.g. built_value).
@@ -39,37 +41,37 @@ module Fastlane
           Helper::FlutterGenerateBuildRunnerHelper.build
         end
 
-        if params[:coverage_all_imports]
-          UI.message("Generating #{ALL_IMPORTS_TEST_FILE} for coverage...")
-
-          dart_file_literals = Dir['lib/**/*.dart'].reject do |file_name|
-            # ".g.dart" files often are "part of" files and can not be imported
-            # directly. Commonly coverage for generated files is not that useful
-            file_name.end_with?('.g.dart')
-          end.map do |file_name|
-            Helper::FlutterHelper.
-              import_path_for_test(file_name, '..').
-              gsub("'", "\\\\'")
-          end
-
-          File.write(
-            ALL_IMPORTS_TEST_FILE,
-            <<-DART
-// This file is autogenerated by fastlane flutter_generate action.
-// It imports all files in lib/ so that test coverage in percentage
-// of overall project is calculated correctly. Do not modify this
-// file manually!
-
-// ignore_for_file: unused_import, directives_ordering
-// ignore_for_file: avoid_relative_lib_imports
-// ignore_for_file: lines_longer_than_80_chars
-
-#{dart_file_literals.map { |fn| "import '#{fn}';" }.sort.join("\n")}
-
-void main() {}
-            DART
-          )
+        return unless params[:coverage_all_imports]
+
+        UI.message("Generating #{ALL_IMPORTS_TEST_FILE} for coverage...")
+
+        dart_file_literals = Dir['lib/**/*.dart'].reject do |file_name|
+          # ".g.dart" files often are "part of" files and can not be imported
+          # directly. Commonly coverage for generated files is not that useful
+          file_name.end_with?('.g.dart')
+        end.map do |file_name|
+          Helper::FlutterHelper
+            .import_path_for_test(file_name, '..')
+            .gsub("'", "\\\\'")
         end
+
+        File.write(
+          ALL_IMPORTS_TEST_FILE,
+          <<~DART
+            // This file is autogenerated by fastlane flutter_generate action.
+            // It imports all files in lib/ so that test coverage in percentage
+            // of overall project is calculated correctly. Do not modify this
+            // file manually!
+
+            // ignore_for_file: unused_import, directives_ordering
+            // ignore_for_file: avoid_relative_lib_imports
+            // ignore_for_file: lines_longer_than_80_chars
+
+            #{dart_file_literals.map { |fn| "import '#{fn}';" }.sort.join("\n")}
+
+            void main() {}
+          DART
+        )
       end
 
       def self.generate_translation?
@@ -77,15 +79,15 @@ void main() {}
       end
 
       def self.description
-        "(1) Run `flutter packages get`;  " \
-        "(2) Run `build_runner build` if build_runner is in dev_dependencies;" \
-        "   " \
-        "(3) According to `package:intl`, take `$strings_file` and generate " \
-        "`${strings_file.dirname}/arb/intl_messages.arb`, then take all " \
-        "files matching `${strings_file.dirname}/intl_*.arb`, fix them and " \
-        "generate .dart files from them.  " \
-        "(4) Generate an empty test importing all files, which would be used " \
-        "to calculate correct full coverage numbers."
+        '(1) Run `flutter packages get`;  ' \
+        '(2) Run `build_runner build` if build_runner is in dev_dependencies;' \
+        '   ' \
+        '(3) According to `package:intl`, take `$strings_file` and generate ' \
+        '`${strings_file.dirname}/arb/intl_messages.arb`, then take all ' \
+        'files matching `${strings_file.dirname}/intl_*.arb`, fix them and ' \
+        'generate .dart files from them.  ' \
+        '(4) Generate an empty test importing all files, which would be used ' \
+        'to calculate correct full coverage numbers.'
       end
 
       def self.available_options
@@ -96,19 +98,15 @@ void main() {}
             env_name: 'FL_FLUTTER_INTL_STRINGS_FILE',
             description: 'Path to source .dart file with Intl.message calls',
             verify_block: proc do |value|
-              if generate_translation?
-                unless File.exist?(value)
-                  UI.user_error!("File `#{value}' does not exist")
-                end
-              end
+              UI.user_error!("File `#{value}' does not exist") if generate_translation? && !File.exist?(value)
             end,
-            default_value: 'lib/intl/intl.dart',
+            default_value: 'lib/intl/intl.dart'
           ),
           FastlaneCore::ConfigItem.new(
             key: :intl_strings_locale,
             env_name: 'FL_FLUTTER_INTL_STRINGS_LOCALE',
             description: 'Locale of the default data in the strings_file',
-            optional: true,
+            optional: true
           ),
           FastlaneCore::ConfigItem.new(
             key: :coverage_all_imports,
@@ -121,8 +119,8 @@ void main() {}
             to .gitignore!
             DESCRIPTION
             optional: true,
-            type: Boolean,
-          ),
+            type: Boolean
+          )
         ]
       end
     end

data/lib/fastlane/plugin/flutter/base/flutter_action_base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'fastlane/action'
 
 module Fastlane

data/lib/fastlane/plugin/flutter/helper/flutter_bootstrap_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'fastlane_core/ui/ui'
 require 'fileutils'
 
@@ -10,6 +12,7 @@ module Fastlane
           license_file = File.join(licenses_directory, license)
           next if File.exist?(license_file) &&
                   File.readlines(license_file).map(&:strip).include?(hash)
+
           UI.message("Updating Android SDK license in #{license_file}...")
           File.open(license_file, 'a') { |f| f.puts('', hash) }
         end

data/lib/fastlane/plugin/flutter/helper/flutter_generate_build_runner_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'yaml'
 
 module Fastlane
@@ -5,7 +7,7 @@ module Fastlane
     class FlutterGenerateBuildRunnerHelper
       def self.build
         Helper::FlutterHelper.flutter(
-          *%w(packages pub run build_runner build --delete-conflicting-outputs),
+          *%w[packages pub run build_runner build --delete-conflicting-outputs]
         )
       end
     end

data/lib/fastlane/plugin/flutter/helper/flutter_generate_intl_helper.rb

@@ -1,6 +1,7 @@
+# frozen_string_literal: true
+
 require 'fileutils'
 require 'json'
-require 'set'
 require 'yaml'
 
 require_relative 'flutter_helper'
@@ -10,101 +11,101 @@ module Fastlane
   end
 end
 
-class Fastlane::Helper::FlutterGenerateIntlHelper
-  def self.generate(messages_filename, messages_locale = nil)
-    dart_files_dirname = File.dirname(messages_filename)
-    arb_files_dirname = File.join(dart_files_dirname, 'arb')
-    full_arb_filename = generate_arb_from_dart(
-      messages_filename, messages_locale, arb_files_dirname
-    )
+module Fastlane
+  module Helper
+    class FlutterGenerateIntlHelper
+      def self.generate(messages_filename, messages_locale = nil)
+        dart_files_dirname = File.dirname(messages_filename)
+        arb_files_dirname = File.join(dart_files_dirname, 'arb')
+        full_arb_filename = generate_arb_from_dart(
+          messages_filename, messages_locale, arb_files_dirname
+        )
 
-    arb_filenames = amend_arb_files(arb_files_dirname, full_arb_filename)
+        arb_filenames = amend_arb_files(arb_files_dirname, full_arb_filename)
 
-    unless messages_locale
-      # Don't generate .dart for the ARB generated from original, unless it
-      # has its own locale.
-      arb_filenames.delete(full_arb_filename)
-    end
+        unless messages_locale
+          # Don't generate .dart for the ARB generated from original, unless it
+          # has its own locale.
+          arb_filenames.delete(full_arb_filename)
+        end
 
-    Fastlane::UI.message('Generating .dart files from .arb...')
-    Fastlane::Helper::FlutterHelper.flutter(
-      *%W(packages pub run intl_translation:generate_from_arb
-          --output-dir=#{dart_files_dirname}
-          --no-use-deferred-loading
-          #{messages_filename}) + arb_filenames
-    )
-  end
+        Fastlane::UI.message('Generating .dart files from .arb...')
+        Fastlane::Helper::FlutterHelper.flutter(
+          *%W[packages pub run intl_translation:generate_from_arb
+              --output-dir=#{dart_files_dirname}
+              --no-use-deferred-loading
+              #{messages_filename}] + arb_filenames
+        )
+      end
 
-  def self.amend_arb_files(arb_files_dirname, full_arb_filename)
-    full_arb_json = JSON.parse(File.read(full_arb_filename))
-
-    # Sort files for consistency, because generated messages_all.dart will
-    # have imports ordered as in the command line below.
-    arb_filenames = Dir.glob(File.join(arb_files_dirname, 'intl_*.arb')).sort
-    arb_filenames.each do |arb_filename|
-      arb_json = JSON.parse(File.read(arb_filename))
-      if arb_filename != full_arb_filename
-        Fastlane::UI.message("Amending #{arb_filename}...")
-        full_arb_json.each_pair do |k, v|
-          # Ignore @@keys. We don't want to copy @@locale over all files, and
-          # it's often unspecified to be inferred from file name.
-          arb_json[k] ||= v unless k.start_with?('@@')
+      def self.amend_arb_files(arb_files_dirname, full_arb_filename)
+        full_arb_json = JSON.parse(File.read(full_arb_filename))
+
+        # Sort files for consistency, because generated messages_all.dart will
+        # have imports ordered as in the command line below.
+        arb_filenames = Dir.glob(File.join(arb_files_dirname, 'intl_*.arb')).sort
+        arb_filenames.each do |arb_filename|
+          arb_json = JSON.parse(File.read(arb_filename))
+          if arb_filename != full_arb_filename
+            Fastlane::UI.message("Amending #{arb_filename}...")
+            full_arb_json.each_pair do |k, v|
+              # Ignore @@keys. We don't want to copy @@locale over all files, and
+              # it's often unspecified to be inferred from file name.
+              arb_json[k] ||= v unless k.start_with?('@@')
+            end
+            arb_json.keep_if { |k| full_arb_json.key?(k) }
+          end
+          File.write(arb_filename, "#{JSON.pretty_generate(arb_json)}\n")
         end
-        arb_json.keep_if { |k| full_arb_json.key?(k) }
       end
-      File.write(arb_filename, JSON.pretty_generate(arb_json) + "\n")
-    end
-  end
 
-  def self.generate_arb_from_dart(dart_filename, dart_locale, arb_dirname)
-    arb_filename = File.join(arb_dirname, 'intl_messages.arb')
-    Fastlane::UI.message("Generating #{arb_filename} from #{dart_filename}...")
+      def self.generate_arb_from_dart(dart_filename, dart_locale, arb_dirname)
+        arb_filename = File.join(arb_dirname, 'intl_messages.arb')
+        Fastlane::UI.message("Generating #{arb_filename} from #{dart_filename}...")
 
-    if File.exist?(arb_filename)
-      arb_file_was = File.read(arb_filename)
-    else
-      # The file may not exist on the first run. Then it's also probable that
-      # the output directory does not exist yet.
-      FileUtils.mkdir_p(arb_dirname)
-    end
+        if File.exist?(arb_filename)
+          arb_file_was = File.read(arb_filename)
+        else
+          # The file may not exist on the first run. Then it's also probable that
+          # the output directory does not exist yet.
+          FileUtils.mkdir_p(arb_dirname)
+        end
 
-    extract_to_arb_options = ["--output-dir=#{arb_dirname}"]
-    if dart_locale
-      extract_to_arb_options.push("--locale=#{dart_locale}")
-    end
+        extract_to_arb_options = ["--output-dir=#{arb_dirname}"]
+        extract_to_arb_options.push("--locale=#{dart_locale}") if dart_locale
 
-    Fastlane::Helper::FlutterHelper.flutter(
-      *%w(packages pub run intl_translation:extract_to_arb),
-      *extract_to_arb_options, dart_filename
-    )
-
-    # intl will update @@last_modified even if there are no updates; this
-    # leaves Git directory unnecessary dirty. If that's the only change,
-    # just restore the previous contents.
-    if arb_file_was
-      if restore_last_modified(arb_filename, arb_file_was)
-        Fastlane::UI.message(
-          "@@last_modified has been restored in #{arb_filename}"
+        Fastlane::Helper::FlutterHelper.flutter(
+          *%w[packages pub run intl_translation:extract_to_arb],
+          *extract_to_arb_options, dart_filename
         )
+
+        # intl will update @@last_modified even if there are no updates; this
+        # leaves Git directory unnecessary dirty. If that's the only change,
+        # just restore the previous contents.
+        if arb_file_was && restore_last_modified(arb_filename, arb_file_was)
+          Fastlane::UI.message(
+            "@@last_modified has been restored in #{arb_filename}"
+          )
+        end
+
+        arb_filename
       end
-    end
 
-    arb_filename
-  end
+      def self.restore_last_modified(filename, old_content)
+        new_content_tree = JSON.parse(File.read(filename))
+        old_content_tree = JSON.parse(old_content)
+        new_content_tree['@@last_modified'] = old_content_tree['@@last_modified']
+
+        # Use to_json to compare the objects deep and in consistent format.
+        if new_content_tree.to_json == old_content_tree.to_json
+          # Except for the @@last_modified attribute that we replaced
+          # above, the objects are identical. Restore previous timestamp.
+          File.write(filename, old_content)
+          return true
+        end
 
-  def self.restore_last_modified(filename, old_content)
-    new_content_tree = JSON.parse(File.read(filename))
-    old_content_tree = JSON.parse(old_content)
-    new_content_tree['@@last_modified'] = old_content_tree['@@last_modified']
-
-    # Use to_json to compare the objects deep and in consistent format.
-    if new_content_tree.to_json == old_content_tree.to_json
-      # Except for the @@last_modified attribute that we replaced
-      # above, the objects are identical. Restore previous timestamp.
-      File.write(filename, old_content)
-      return true
+        false
+      end
     end
-
-    false
   end
 end

data/lib/fastlane/plugin/flutter/helper/flutter_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'shellwords'
 require 'yaml'
 
@@ -28,10 +30,10 @@ module Fastlane
           elsif ENV.include?('FLUTTER_ROOT')
             # FLUTTER_ROOT is a standard environment variable from Flutter.
             UI.message("Determined Flutter location as #{ENV['FLUTTER_ROOT']}" \
-              " because environment variable FLUTTER_ROOT points there" \
+              ' because environment variable FLUTTER_ROOT points there' \
               " (current directory is #{Dir.pwd}).")
             ENV['FLUTTER_ROOT']
-          elsif flutter_binary = FastlaneCore::CommandExecutor.which('flutter')
+          elsif (flutter_binary = FastlaneCore::CommandExecutor.which('flutter'))
             location = File.dirname(File.dirname(flutter_binary))
             UI.message("Determined Flutter location as #{location} because"\
               " 'flutter' executable in PATH is located there" \
@@ -39,9 +41,9 @@ module Fastlane
             location
           else
             # Where we'd prefer to install flutter.
-            UI.message("Determined desired Flutter location as" \
+            UI.message('Determined desired Flutter location as' \
               " #{pinned_flutter_path} because that's where this fastlane" \
-              " plugin would install Flutter by default.")
+              ' plugin would install Flutter by default.')
             pinned_flutter_path
           end
         )
@@ -65,9 +67,7 @@ module Fastlane
       end
 
       def self.import_path_for_test(file_to_import, relative_path)
-        unless file_to_import.start_with?('lib/')
-          return File.join(relative_path, file_to_import)
-        end
+        return File.join(relative_path, file_to_import) unless file_to_import.start_with?('lib/')
 
         # Import file schema in tests have to match files in lib/ exactly. From
         # Dart perspective, symbols in files imported via relative and
@@ -78,7 +78,7 @@ module Fastlane
           # chances are, it's using "package:..." imports. Indeed, checking the
           # file itself isn't sufficient to explore all of its dependencies, but
           # we expect imports to be consistent in the project.
-          "#{package_specification}#{file_to_import['lib/'.size..-1]}"
+          "#{package_specification}#{file_to_import['lib/'.size..]}"
         else
           File.join(relative_path, file_to_import)
         end
@@ -102,14 +102,14 @@ module Fastlane
           end
 
           unless wait_thread.value.success? || (ignore_error == true)
-            UI.shell_error!(<<-ERROR)
-The following command has failed:
+            UI.shell_error!(<<~ERROR)
+              The following command has failed:
 
-$ #{command}
-[#{wait_thread.value}]
+              $ #{command}
+              [#{wait_thread.value}]
 
-#{errors_thread.value}
-ERROR
+              #{errors_thread.value}
+            ERROR
           end
 
           ignore_error

data/lib/fastlane/plugin/flutter/version.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Fastlane
   module Flutter
-    VERSION = '0.7.1'
+    VERSION = '0.8.0'
   end
 end

data/lib/fastlane/plugin/flutter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'fastlane/plugin/flutter/version'
 
 module Fastlane

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: fastlane-plugin-flutter
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.8.0
 platform: ruby
 authors:
 - Artem Sheremet
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-02-01 00:00:00.000000000 Z
+date: 2024-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -25,7 +25,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: codecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -39,7 +39,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: fastlane
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.91.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.91.0
+- !ruby/object:Gem::Dependency
+  name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +67,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec_junit_formatter
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,7 +81,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec_junit_formatter
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -84,18 +112,18 @@ dependencies:
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.49.1
+        version: '1.64'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.49.1
+        version: '1.64'
 - !ruby/object:Gem::Dependency
-  name: rubocop-require_tools
+  name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,7 +137,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: simplecov
+  name: rubocop-require_tools
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -123,7 +151,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: codecov
+  name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -137,20 +165,20 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: fastlane
+  name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.91.0
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.91.0
-description: 
+        version: '0'
+description:
 email: artem@sheremet.ch
 executables: []
 extensions: []
@@ -173,7 +201,7 @@ homepage: https://github.com/dotdoom/fastlane-plugin-flutter
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -188,8 +216,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.5.3
+signing_key:
 specification_version: 4
 summary: Flutter actions plugin for Fastlane
 test_files: []