jekyll-l10n 1.3.0 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7fa24f781e7892870dbdd97d38606b511024be5d0bf026ff4f9fca0b4d680738
-  data.tar.gz: 298e9757f9dd189fdc41d5c545e019c4f2a82a607512944efa31ac61df639100
+  metadata.gz: e9b061ddf1f9f9072efe9135d023d05d55d7dd8545bd34988ecae6f59e03a2e2
+  data.tar.gz: 6c8704fa0e33e96c8e9918d5f7539db8203f291b4a2608e8946846e86fea8a94
 SHA512:
-  metadata.gz: 6153450cfcf936014afac2e2c96aa7d6be1b6995de8c03c7551a329bdad32ae7f4b353bd82068e14aa64b6612bfdfa0f69a53bfabcf850a5401e43f097bf02c8
-  data.tar.gz: a366a109b99e560a8ec305a1a0db3807dbdc903d2a45007f47382d2a5f7a19a13ce8de7b7eb2010a933bcd9ac72b2016e49e65c890c7b98e29eea6602e9c420c
+  metadata.gz: 709ab40d1a6f0ada0810751f54279d4c466d00349a46a9c7916d4a705289af6e8c142ebbd31fefd520c1205089f1f662354f702b381fe8bb44129ab4fde157fc
+  data.tar.gz: acda3c29b32433e2c3b1e90004690955b0da0219a8c896f8cfc28aa61e2ff190e238352e08e35cd606c4717524b8d6c731cd3efac673be115ec89ce89a9fdb32

data/lib/jekyll-l10n/constants.rb

@@ -22,27 +22,27 @@ module Jekyll
       # Matches ISO 639-1 (2 letter language) with optional ISO 3166-1 (2 letter country)
       # Examples: 'en', 'es', 'fr', 'pt_BR', 'zh_CN'
       # @return [Regexp]
-      LOCALE_PATTERN = %r!^[a-z]{2}(_[A-Z]{2})?$!.freeze
+      LOCALE_PATTERN = /^[a-z]{2}(_[A-Z]{2})?$/.freeze
 
       # ## Translation Fallback Modes
 
       # Fallback mode: use original English text if translation not found
       # @return [String] "english"
-      FALLBACK_MODE_ENGLISH = "english"
+      FALLBACK_MODE_ENGLISH = 'english'
 
       # Fallback mode: wrap untranslated text with markers (e.g., "[UNTRANSLATED] text")
       # @return [String] "marker"
-      FALLBACK_MODE_MARKER = "marker"
+      FALLBACK_MODE_MARKER = 'marker'
 
       # Fallback mode: leave text blank if no translation found
       # @return [String] "empty"
-      FALLBACK_MODE_EMPTY = "empty"
+      FALLBACK_MODE_EMPTY = 'empty'
 
       # ## Translation Markers
 
       # Marker used to indicate untranslated strings in marker fallback mode
       # @return [String] "[UNTRANSLATED]"
-      UNTRANSLATED_MARKER = "[UNTRANSLATED]"
+      UNTRANSLATED_MARKER = '[UNTRANSLATED]'
 
       # ## PO File Formatting (GNU Gettext Standard)
 
@@ -82,7 +82,7 @@ module Jekyll
 
       # Default directory for storing PO translation files
       # @return [String] "_locales"
-      DEFAULT_LOCALES_DIR = "_locales"
+      DEFAULT_LOCALES_DIR = '_locales'
 
       # Default fallback mode when translation is not found
       # @return [String] "english"
@@ -90,14 +90,14 @@ module Jekyll
 
       # Default HTML attributes to extract from elements (can be overridden per-page)
       # @return [Array<String>] ["title", "alt", "aria-label", "placeholder", "aria-description"]
-      DEFAULT_TRANSLATABLE_ATTRIBUTES = %w(title alt aria-label placeholder aria-description).freeze
+      DEFAULT_TRANSLATABLE_ATTRIBUTES = %w[title alt aria-label placeholder aria-description].freeze
 
       # ## LibreTranslate Integration Defaults
 
       # Default LibreTranslate API endpoint URL
       # Uses localhost:5000 as default for local development
       # @return [String] "http://localhost:5000"
-      DEFAULT_LIBRETRANSLATE_API_URL = "http://localhost:5000"
+      DEFAULT_LIBRETRANSLATE_API_URL = 'http://localhost:5000'
 
       # Default timeout (in seconds) for LibreTranslate API requests
       # @return [Integer] 300 seconds (5 minutes)
@@ -130,12 +130,12 @@ module Jekyll
       # Default source locale for LibreTranslate API
       # The language of the original content being translated
       # @return [String] "en" (English)
-      DEFAULT_LIBRETRANSLATE_SOURCE_LOCALE = "en"
+      DEFAULT_LIBRETRANSLATE_SOURCE_LOCALE = 'en'
 
       # Default text format for LibreTranslate API requests
       # Either 'text' (plain text) or 'html' (preserves markup)
       # @return [String] "html"
-      DEFAULT_LIBRETRANSLATE_FORMAT = "html"
+      DEFAULT_LIBRETRANSLATE_FORMAT = 'html'
     end
   end
 end

data/lib/jekyll-l10n/extraction/compendium_merger.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require_relative "../po_file/manager"
-require_relative "../utils/page_locales_config"
-require_relative "../utils/site_config_accessor"
+require_relative '../po_file/manager'
+require_relative '../utils/page_locales_config'
+require_relative '../utils/site_config_accessor'
 
 module Jekyll
   module L10n
@@ -34,7 +34,7 @@ module Jekyll
       def initialize(site)
         @site = site
         with_locales_data = SiteConfigAccessor.extract_locales_data(@site)
-        @site_config = PageLocalesConfig.new({ "with_locales_data" => with_locales_data })
+        @site_config = PageLocalesConfig.new({ 'with_locales_data' => with_locales_data })
       end
 
       # Merge page-specific PO files into compendia for all locales.
@@ -55,6 +55,7 @@ module Jekyll
       private
 
       # Process a single locale: merge existing compendium with new page translations
+      # Skips writing when translations are unchanged to avoid unnecessary file rewrites
       def process_locale(locale, po_manager, config)
         compendium_path = File.join(@site.source, config.locales_dir, "#{locale}.po")
         existing_compendium = load_existing_compendium(compendium_path)
@@ -62,12 +63,27 @@ module Jekyll
 
         combined = build_combined_hash(existing_compendium)
         merge_into_combined(combined, merged)
-        combined_entries = format_compendium_entries(combined)
 
-        po_manager.save_compendium(locale, combined_entries)
+        unless compendium_unchanged?(existing_compendium, combined)
+          combined_entries = format_compendium_entries(combined)
+          po_manager.save_compendium(locale, combined_entries)
+        end
         cleanup_locale_directory(locale, config)
       end
 
+      # Check if compendium content is unchanged (same msgids and msgstrs)
+      def compendium_unchanged?(existing_compendium, combined)
+        return false if existing_compendium.empty? && !combined.empty?
+
+        normalized_existing = build_combined_hash(existing_compendium)
+        return false unless normalized_existing.keys.sort == combined.keys.sort
+
+        combined.all? do |msgid, data|
+          existing = normalized_existing[msgid]
+          existing && existing[:msgstr] == data[:msgstr]
+        end
+      end
+
       # Load existing compendium translations or return empty hash if not found
       def load_existing_compendium(compendium_path)
         if File.exist?(compendium_path)
@@ -89,9 +105,9 @@ module Jekyll
       # Normalize entry format to ensure consistent hash structure with :msgstr and :reference keys
       def normalize_compendium_entry(data)
         if data.is_a?(Hash)
-          { :msgstr => data[:msgstr], :reference => data[:reference] }
+          { msgstr: data[:msgstr], reference: data[:reference] }
         else
-          { :msgstr => data, :reference => nil }
+          { msgstr: data, reference: nil }
         end
       end
 
@@ -108,20 +124,24 @@ module Jekyll
 
       # Update reference for existing entry if new reference is available
       def update_entry_reference(combined_entry, entry)
-        if combined_entry[:reference].nil? && entry.is_a?(Hash) && entry[:reference]
-          combined_entry[:reference] = entry[:reference]
-        end
+        return unless combined_entry[:reference].nil? && entry.is_a?(Hash) && entry[:reference]
+
+        combined_entry[:reference] = entry[:reference]
       end
 
-      # Create new entry for untranslated string with optional reference
+      # Create new entry preserving any existing translation and reference
       def create_new_entry(entry)
-        { :msgstr => "", :reference => entry.is_a?(Hash) ? entry[:reference] : nil }
+        if entry.is_a?(Hash)
+          { msgstr: entry[:msgstr] || '', reference: entry[:reference] }
+        else
+          { msgstr: '', reference: nil }
+        end
       end
 
       # Convert combined hash to array of entries suitable for PO file writing
       def format_compendium_entries(combined)
         combined.map do |msgid, data|
-          entry = { :msgid => msgid, :msgstr => data[:msgstr] }
+          entry = { msgid: msgid, msgstr: data[:msgstr] }
           entry[:reference] = data[:reference] if data[:reference]
           entry
         end
@@ -134,9 +154,10 @@ module Jekyll
         FileUtils.rm_rf(locale_dir) if File.directory?(locale_dir)
       end
 
-      private :process_locale, :load_existing_compendium, :build_combined_hash,
-              :normalize_compendium_entry, :merge_into_combined, :update_entry_reference,
-              :create_new_entry, :format_compendium_entries, :cleanup_locale_directory
+      private :process_locale, :compendium_unchanged?, :load_existing_compendium,
+              :build_combined_hash, :normalize_compendium_entry, :merge_into_combined,
+              :update_entry_reference, :create_new_entry, :format_compendium_entries,
+              :cleanup_locale_directory
     end
   end
 end

data/lib/jekyll-l10n/extraction/compendium_translator.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-require_relative "../po_file/manager"
-require_relative "../utils/page_locales_config"
-require_relative "../utils/po_entry_converter"
-require_relative "../utils/site_config_accessor"
-require_relative "../utils/logger_formatter"
-require_relative "../translation/libre_translator"
+require_relative '../po_file/manager'
+require_relative '../utils/page_locales_config'
+require_relative '../utils/po_entry_converter'
+require_relative '../utils/site_config_accessor'
+require_relative '../utils/logger_formatter'
+require_relative '../translation/libre_translator'
 
 module Jekyll
   module L10n
@@ -34,7 +34,7 @@ module Jekyll
       def initialize(site)
         @site = site
         with_locales_data = SiteConfigAccessor.extract_locales_data(@site)
-        @site_config = PageLocalesConfig.new({ "with_locales_data" => with_locales_data })
+        @site_config = PageLocalesConfig.new({ 'with_locales_data' => with_locales_data })
       end
 
       # Translate compendia for all configured locales.
@@ -68,7 +68,7 @@ module Jekyll
 
       def process_single_locale(locale, config, translator, po_manager)
         compendium_path = File.join(@site.source, config.locales_dir, "#{locale}.po")
-        LoggerFormatter.debug_if_enabled("CompendiumTranslator",
+        LoggerFormatter.debug_if_enabled('CompendiumTranslator',
                                          "Processing compendium file: #{compendium_path}")
         return unless File.exist?(compendium_path)
 
@@ -86,29 +86,29 @@ module Jekyll
       end
 
       def log_compendia_enabled_check(enabled)
-        msg = "translate_compendia_for_locale called, libretranslate_enabled:"
-        LoggerFormatter.debug_if_enabled("CompendiumTranslator", "#{msg} #{enabled}")
+        msg = 'translate_compendia_for_locale called, libretranslate_enabled:'
+        LoggerFormatter.debug_if_enabled('CompendiumTranslator', "#{msg} #{enabled}")
       end
 
       def log_compendium_stats(locale, po_entries, compendium_path)
         msg = "Locale #{locale}: Loaded #{po_entries.length} entries from compendium"
-        LoggerFormatter.debug_if_enabled("CompendiumTranslator", "#{msg} #{compendium_path}")
+        LoggerFormatter.debug_if_enabled('CompendiumTranslator', "#{msg} #{compendium_path}")
 
         empty_count = count_empty_entries(po_entries)
         LoggerFormatter.debug_if_enabled(
-          "CompendiumTranslator",
+          'CompendiumTranslator',
           "Locale #{locale}: #{empty_count} empty msgstr entries to translate"
         )
       end
 
       def log_translation_complete_for_locale(locale)
         msg = "Locale #{locale}: Translation complete, saving compendium"
-        LoggerFormatter.debug_if_enabled("CompendiumTranslator", msg)
+        LoggerFormatter.debug_if_enabled('CompendiumTranslator', msg)
       end
 
       def log_compendium_saved(locale, compendium_path)
         msg = "Locale #{locale}: Saved compendium to"
-        LoggerFormatter.debug_if_enabled("CompendiumTranslator", "#{msg} #{compendium_path}")
+        LoggerFormatter.debug_if_enabled('CompendiumTranslator', "#{msg} #{compendium_path}")
       end
 
       def count_empty_entries(po_entries)
@@ -118,14 +118,14 @@ module Jekyll
       end
 
       def log_translation_start(config)
-        locales = config.locales.join(", ")
-        Jekyll.logger.info "Localization",
+        locales = config.locales.join(', ')
+        Jekyll.logger.info 'Localization',
                            "Starting LibreTranslate translation for locales: #{locales}"
       end
 
       def log_translation_complete(config)
-        locales = config.locales.join(", ")
-        Jekyll.logger.info "Localization",
+        locales = config.locales.join(', ')
+        Jekyll.logger.info 'Localization',
                            "LibreTranslate translation complete for locales: #{locales}"
       end
 

data/lib/jekyll-l10n/extraction/config_loader.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative "../utils/page_locales_config"
+require_relative '../utils/page_locales_config'
 
 module Jekyll
   module L10n
@@ -69,10 +69,10 @@ module Jekyll
       # @return [Hash, nil] Page front matter data if found, nil otherwise
       def find_page_config_for_file(file_path)
         @site.pages.each do |page|
-          next unless page.data["with_locales"] == true
+          next unless page.data['with_locales'] == true
 
-          page_output = page.output_ext ? page.destination("") : page.destination("/")
-          next unless file_path.end_with?(page_output.sub(%r!/$!, "/index.html"))
+          page_output = page.output_ext ? page.destination('') : page.destination('/')
+          next unless file_path.end_with?(page_output.sub(%r{/$}, '/index.html'))
 
           return page.data
         end
@@ -88,10 +88,10 @@ module Jekyll
       # @param file_path [String] Path to file to check
       # @return [Boolean] True if file is in a locale subdirectory
       def skip_localized_page?(file_path)
-        relative_path = file_path.sub(@dest, "")
+        relative_path = file_path.sub(@dest, '')
 
         all_locales = @site.pages.map do |p|
-          p.data.dig("with_locales_data", "locales") || []
+          p.data.dig('with_locales_data', 'locales') || []
         end
         all_locales.flatten!
         all_locales.uniq!
@@ -106,8 +106,8 @@ module Jekyll
       # @param config [Hash] Page front matter data
       # @return [Array<String>] CSS selectors for excluded elements
       def extract_exclude_selectors(config)
-        config.data.dig("with_locales_data", "extraction", "exclude_selectors") ||
-          ["script", "style", "code.language-plaintext", "pre code"]
+        config.data.dig('with_locales_data', 'extraction', 'exclude_selectors') ||
+          ['script', 'style', 'code.language-plaintext', 'pre code']
       end
     end
   end

data/lib/jekyll-l10n/extraction/dom_attribute_extractor.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative "../utils/xpath_reference_generator"
-require_relative "../utils/text_validator"
+require_relative '../utils/xpath_reference_generator'
+require_relative '../utils/text_validator'
 
 module Jekyll
   module L10n
@@ -24,7 +24,7 @@ module Jekyll
     #                                           ['title', 'alt', 'aria-label'])
     #   # Returns array of extraction entries for all valid attribute values
     module DomAttributeExtractor
-      extend self
+      module_function
 
       # Extract attribute values from an HTML element.
       #
@@ -47,7 +47,7 @@ module Jekyll
         attrs = extract_node_attributes(node, translatable_attrs)
         attrs.map do |attr_text, attr_name|
           reference = XPathReferenceGenerator.generate(node, file_path, dest, attr_name)
-          { :msgid => attr_text, :msgstr => "", :reference => reference }
+          { msgid: attr_text, msgstr: '', reference: reference }
         end
       end
 

data/lib/jekyll-l10n/extraction/dom_text_extractor.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require_relative "../utils/text_normalizer"
-require_relative "../utils/xpath_reference_generator"
-require_relative "../utils/html_text_utils"
-require_relative "../utils/html_elements"
-require_relative "../utils/text_validator"
+require_relative '../utils/text_normalizer'
+require_relative '../utils/xpath_reference_generator'
+require_relative '../utils/html_text_utils'
+require_relative '../utils/html_elements'
+require_relative '../utils/text_validator'
 
 module Jekyll
   module L10n
@@ -48,7 +48,7 @@ module Jekyll
         return nil if text.nil?
 
         reference = XPathReferenceGenerator.generate(node, file_path, dest)
-        { :msgid => text, :msgstr => "", :reference => reference }
+        { msgid: text, msgstr: '', reference: reference }
       end
 
       def extractable?(node)

data/lib/jekyll-l10n/extraction/extractor.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
-require_relative "config_loader"
-require_relative "result_saver"
-require_relative "logger"
-require_relative "html_string_extractor"
-require_relative "../utils/file_operations"
-require_relative "../utils/site_config_accessor"
-require_relative "../utils/error_handler"
+require_relative 'config_loader'
+require_relative 'result_saver'
+require_relative 'logger'
+require_relative 'html_string_extractor'
+require_relative '../utils/file_operations'
+require_relative '../utils/site_config_accessor'
+require_relative '../utils/error_handler'
 
 module Jekyll
   module L10n
@@ -72,7 +72,7 @@ module Jekyll
       #   result = extractor.extract_site
       #   puts "Processed #{result[:files_processed]} files"
       def extract_site
-        Jekyll.logger.info "Localization", "Extracting translatable strings..."
+        Jekyll.logger.info 'Localization', 'Extracting translatable strings...'
         start_time = Time.now
         stats = process_all_html_files
         translate_all_compendia
@@ -81,8 +81,8 @@ module Jekyll
       end
 
       def process_all_html_files
-        stats = { :files_processed => 0, :strings_extracted => 0, :po_files_created => 0 }
-        html_files = Dir.glob(File.join(@dest, "**", "*.html"))
+        stats = { files_processed: 0, strings_extracted: 0, po_files_created: 0 }
+        html_files = Dir.glob(File.join(@dest, '**', '*.html'))
 
         html_files.each do |file_path|
           next if @config_loader.skip_localized_page?(file_path)
@@ -100,7 +100,7 @@ module Jekyll
         config = find_libretranslate_config
         return unless config
 
-        ErrorHandler.handle_with_logging("machine translation") do
+        ErrorHandler.handle_with_logging('machine translation') do
           @result_saver.translate_compendia(config)
         end
       end
@@ -120,7 +120,7 @@ module Jekyll
       end
 
       def default_stats
-        { :files_processed => 0, :strings_extracted => 0, :po_files_created => 0 }
+        { files_processed: 0, strings_extracted: 0, po_files_created: 0 }
       end
 
       def extract_strings_from_file(file_path, config)
@@ -137,9 +137,9 @@ module Jekyll
         return nil unless @site.respond_to?(:pages)
 
         @site.pages.each do |page|
-          next unless page.data["with_locales"] == true
+          next unless page.data['with_locales'] == true
 
-          config = @config_loader.load_page_config(page.destination(""))
+          config = @config_loader.load_page_config(page.destination(''))
           return config if config.libretranslate_enabled?
         end
 
@@ -149,7 +149,7 @@ module Jekyll
       private
 
       def construct_page_path(file_path)
-        file_path.sub("#{@dest}/", "")
+        file_path.sub("#{@dest}/", '')
       end
     end
   end

data/lib/jekyll-l10n/extraction/html_string_extractor.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require_relative "dom_text_extractor"
-require_relative "dom_attribute_extractor"
-require_relative "logger"
+require_relative 'dom_text_extractor'
+require_relative 'dom_attribute_extractor'
+require_relative 'logger'
 
 module Jekyll
   module L10n
@@ -92,10 +92,10 @@ module Jekyll
 
         @exclude_selectors.any? { |selector| node.matches?(selector) }
       rescue Nokogiri::CSS::SyntaxError => e
-        Jekyll.logger.warn "Localization", "CSS selector syntax error: #{e.message}"
+        Jekyll.logger.warn 'Localization', "CSS selector syntax error: #{e.message}"
         false
       rescue StandardError => e
-        Jekyll.logger.warn "Localization", "Selector matching error (continuing): #{e.message}"
+        Jekyll.logger.warn 'Localization', "Selector matching error (continuing): #{e.message}"
         false
       end
     end

data/lib/jekyll-l10n/extraction/logger.rb

@@ -22,7 +22,7 @@ module Jekyll
       # @param error [StandardError] The error that occurred
       # @return [void]
       def self.log_error(file_path, error)
-        Jekyll.logger.error "Localization", "Error extracting from #{file_path}: #{error.message}"
+        Jekyll.logger.error 'Localization', "Error extracting from #{file_path}: #{error.message}"
       end
 
       # Log extraction completion summary.
@@ -37,11 +37,11 @@ module Jekyll
       # @param elapsed [Float] Time elapsed in seconds
       # @return [void]
       def self.log_summary(stats, elapsed)
-        Jekyll.logger.info "Localization", "Extraction complete:"
-        Jekyll.logger.info "Localization", "  Files processed: #{stats[:files_processed]}"
-        Jekyll.logger.info "Localization", "  Strings extracted: #{stats[:strings_extracted]}"
-        Jekyll.logger.info "Localization", "  PO files created/updated: #{stats[:po_files_created]}"
-        Jekyll.logger.info "Localization", "  Time: #{elapsed.round(2)}s"
+        Jekyll.logger.info 'Localization', 'Extraction complete:'
+        Jekyll.logger.info 'Localization', "  Files processed: #{stats[:files_processed]}"
+        Jekyll.logger.info 'Localization', "  Strings extracted: #{stats[:strings_extracted]}"
+        Jekyll.logger.info 'Localization', "  PO files created/updated: #{stats[:po_files_created]}"
+        Jekyll.logger.info 'Localization', "  Time: #{elapsed.round(2)}s"
       end
     end
   end

data/lib/jekyll-l10n/extraction/result_saver.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-require_relative "../po_file/manager"
-require_relative "../utils/page_locales_config"
-require_relative "../utils/site_config_accessor"
-require_relative "../utils/logger_formatter"
-require_relative "compendium_translator"
-require_relative "compendium_merger"
+require_relative '../po_file/manager'
+require_relative '../utils/page_locales_config'
+require_relative '../utils/site_config_accessor'
+require_relative '../utils/logger_formatter'
+require_relative 'compendium_translator'
+require_relative 'compendium_merger'
 
 module Jekyll
   module L10n
@@ -34,7 +34,7 @@ module Jekyll
       def initialize(site)
         @site = site
         with_locales_data = SiteConfigAccessor.extract_locales_data(@site)
-        @site_config = PageLocalesConfig.new({ "with_locales_data" => with_locales_data })
+        @site_config = PageLocalesConfig.new({ 'with_locales_data' => with_locales_data })
       end
 
       # Save extraction results to PO files.
@@ -51,15 +51,15 @@ module Jekyll
       #   - :strings_extracted [Integer] Number of extracted strings
       #   - :po_files_created [Integer] Number of PO files created/updated
       def save_results(config, entries, page_path)
-        LoggerFormatter.debug_if_enabled("ExtractionResultSaver", "Processing page: #{page_path}")
+        LoggerFormatter.debug_if_enabled('ExtractionResultSaver', "Processing page: #{page_path}")
 
         po_manager = PoFileManager.new(@site, config.locales_dir)
         po_files_created = save_po_files(po_manager, config, entries, page_path)
 
         {
-          :files_processed   => 1,
-          :strings_extracted => entries.length,
-          :po_files_created  => po_files_created,
+          files_processed: 1,
+          strings_extracted: entries.length,
+          po_files_created: po_files_created
         }
       end
 
@@ -81,7 +81,7 @@ module Jekyll
         po_files_created = 0
 
         config.locales.each do |locale|
-          po_files_created += 1 if po_manager.save_po_file(locale, entries, :page_path => page_path)
+          po_files_created += 1 if po_manager.save_po_file(locale, entries, page_path: page_path)
         end
 
         CompendiumMerger.new(@site).merge_compendia(po_manager, config) if config.update_compendium?