linter-compliant version using Time.zone.parse after the .load_rails call

Author: Alistair Davidson

app/lib/mavis_cli/reporting_api/update_vaccination_events.rb

@@ -3,19 +3,24 @@
 module MavisCLI
   module ReportingAPI
     class UpdateVaccinationEvents < Dry::CLI::Command
-      desc "Update ReportingAPI Vaccination Events"
+      desc <<-DESC
+
+      Update ReportingAPI Vaccination Events for all VaccinationRecords created after a given DateTime.
+      DateTimes can be given in any format parsable by Time.parse.
+      Defaults to one year ago.
+      DESC
+
       option :from,
-             desc:
-               "Only consider vaccination records created after this time (any format parsable by Time.parse())",
+             desc: "Only consider vaccination records created after this time",
              type: :string,
              optional: true,
-             aliases: %w[-f --from],
-             default: (Time.zone.now - 365).iso8601
+             aliases: %w[--from],
+             default: nil
 
-      def call(from:, **)
+      def call(from: nil, **)
         MavisCLI.load_rails
 
-        min_datetime = Time.zone.parse(from)
+        min_datetime = from ? Time.zone.parse(from) : (Time.zone.now - 1.year)
 
         records = VaccinationRecord.where("created_at > ?", min_datetime)
         puts "#{records.count} VaccinationRecords created since #{min_datetime.iso8601}"