Only require `simple_po_parser` in rake task that needs it

That way we don't need to install it in production, since it's really not needed there.

Only require `simple_po_parser` in rake task that needs it
7199e882 · Bob Van Landuyt · a3af6830 · 7199e882 · 7199e882 · 7199e882
Commit 7199e882 authored Sep 03, 2017 by Bob Van Landuyt
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

po_linter.rb lib/gitlab/i18n/po_linter.rb +0 -2

gettext.rake lib/tasks/gettext.rake +1 -0

po_linter_spec.rb spec/lib/gitlab/i18n/po_linter_spec.rb +1 -0

No files found.
--- a/lib/gitlab/i18n/po_linter.rb
+++ b/lib/gitlab/i18n/po_linter.rb
-require 'simple_po_parser'
-
 module Gitlab
  module I18n
    class PoLinter

--- a/lib/tasks/gettext.rake
+++ b/lib/tasks/gettext.rake
 require "gettext_i18n_rails/tasks"
+require 'simple_po_parser'

 namespace :gettext do
  # Customize list of translatable files

--- a/spec/lib/gitlab/i18n/po_linter_spec.rb
+++ b/spec/lib/gitlab/i18n/po_linter_spec.rb
 require 'spec_helper'
+require 'simple_po_parser'

 describe Gitlab::I18n::PoLinter do
  let(:linter) { described_class.new(po_path) }