HighLine namespace fix (#177)

* Use HighLine in a way that does not pollute the global namespace * version bump
sparkapi · Jan 3, 2022 · 5f0072a · 5f0072a
1 parent 24fa16d
commit 5f0072a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,3 +1,6 @@
+v1.5.6
+ - Modify usage of HighLine so that it does not pollute the global namespace
+
 v1.5.5
  - Add logging of an errored oauth2 response
 

diff --git a/VERSION b/VERSION
@@ -1 +1 @@
-1.5.5
+1.5.6
diff --git a/lib/spark_api/authentication/oauth2_impl/cli_provider.rb b/lib/spark_api/authentication/oauth2_impl/cli_provider.rb
@@ -1,4 +1,4 @@
-require "highline/import"
+require "highline"
 
 module SparkApi
   module Authentication
@@ -18,7 +18,7 @@ def initialize(credentials)
         def redirect(url)
           puts "Missing OAuth2 session, redirecting..."
           puts "Please visit #{url}, login as a user, and paste the authorization code here:"
-          self.code = ask("Authorization code?") do |q|
+          self.code = HighLine.ask("Authorization code?") do |q|
             q.whitespace = :strip_and_collapse
             q.validate = /^\w+$/
           end