Merge pull request #92 from alphagov/sengi/version

Release version 4.1.0.

CHANGELOG.md

@@ -2,6 +2,14 @@
 
 # Unreleased
 
+Nothing yet.
+
+# 4.1.0
+
+  * Deprecate usage of `Plek.current` this method will be removed in next
+    major version. This adds a warning for users.
+  * Remove public setter methods for `parent_domain` and `external_domain`.
+    These are not used anywhere so there are no API compatibility issues.
   * Allow setting `GOVUK_APP_DOMAIN=""` (empty string). Similarly for
     `GOVUK_APP_DOMAIN_EXTERNAL`. This allows single-label domains to be used in
     service URLs instead of FQDNs, which eliminates a lot of configuration

Gemfile

@@ -1,4 +1,4 @@
-source "http://rubygems.org"
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in plek.gemspec
 gemspec

README.md

@@ -72,4 +72,4 @@ component, for example `frontend` or `content-store`, as opposed to
 
 ## Versioning policy
 
-This is versioned according to [Semantic Versioning 2.0](http://semver.org/)
+This is versioned according to [Semantic Versioning 2.0](https://semver.org/)

lib/plek.rb

@@ -144,7 +144,10 @@ class << self
     #     Plek.current.find('foo')
     # as well as the new style:
     #     Plek.new.find('foo')
-    alias_method :current, :new
+    def current(...)
+      warn "Plek.current is deprecated and will be removed. Use Plek.new or Plek.find instead."
+      new(...)
+    end
 
     # Convenience wrapper.  The same as calling +Plek.new.find+.
     # @see #find

lib/plek/version.rb

@@ -1,3 +1,3 @@
 class Plek
-  VERSION = "4.0.0".freeze
+  VERSION = "4.1.0".freeze
 end

test/plek_test.rb

@@ -75,7 +75,11 @@ def test_should_return_dash_divided_source_in_production
 
   def test_should_be_able_to_use_current_for_old_style_calls
     ClimateControl.modify GOVUK_APP_DOMAIN: "foo.bar.baz" do
-      assert_equal Plek.new.find("foo"), Plek.current.find("foo")
+      old_style = nil
+      assert_output("", "Plek.current is deprecated and will be removed. Use Plek.new or Plek.find instead.\n") do
+        old_style = Plek.current.find("foo")
+      end
+      assert_equal Plek.new.find("foo"), old_style
     end
   end