style fixes

This commit is contained in:
Elizabeth Tackett 2020-06-30 10:23:18 -05:00
parent 198b7ade59
commit 3cae15945b
7 changed files with 19 additions and 18 deletions

View File

@ -4,7 +4,7 @@ require "open3"
module Livecheck
def livecheck_formula_response(formula_name)
puts "- livecheck formula : #{formula_name}"
ohai "- livecheck formula : #{formula_name}"
command_args = [
"brew",
"livecheck",
@ -17,11 +17,12 @@ module Livecheck
end
def parse_livecheck_response(response)
output = response.first.gsub(' ', '').split(/:|==>|\n/)
output = response.first.gsub(" ", "").split(/:|==>|\n/)
# eg: ["burp", "2.2.18", "2.2.18"]
package_name, brew_version, latest_version = output
{'name' => package_name, 'current_brew_version' => brew_version, 'livecheck_latest_version' => latest_version}
{"name" => package_name, "current_brew_version" => brew_version,
"livecheck_latest_version" => latest_version}
end
end

View File

@ -46,14 +46,14 @@ module RepologyParser
end
def validate__repology_packages(outdated_repology_packages, brew_formulas)
ohai "\n---- Verify Outdated Repology Packages as Homebrew Formulae -----"
ohai "\n- Verify Outdated Repology Packages as Homebrew Formulae"
packages = {}
outdated_repology_packages.each do |name, repositories|
# identify homebrew repo
repology_homebrew_repo = repositories.select do
|repo| repo['repo'] == 'homebrew'
|repo| repo["repo"] == "homebrew"
end.first
next if repology_homebrew_repo.empty?
@ -61,7 +61,7 @@ module RepologyParser
#identify latest version amongst repology repos
repositories.each do |repo|
latest_version = repo['version'] if repo['status'] == 'newest'
latest_version = repo["version"] if repo["status"] == "newest"
end
packages[repology_homebrew_repo['srcname']] = {

View File

@ -28,9 +28,9 @@ module Versions
end
def check_for_open_pr(formula_name, download_url)
puts "- Checking for open PRs for formula : #{formula_name}"
ohai "- Checking for open PRs for formula : #{formula_name}"
response = bump_formula_pr(formula_name, download_url)
!response.include? 'Error: These open pull requests may be duplicates'
!response.include? "Error: These open pull requests may be duplicates"
end
end

View File

@ -1,5 +1,5 @@
require_relative 'helpers/parsed_file'
require_relative 'helpers/brew_commands.rb'
require_relative "helpers/parsed_file"
require_relative "helpers/brew_commands.rb"
brew_commands = BrewCommands.new
@ -11,10 +11,10 @@ File.foreach(outdated_pckgs_to_update) do |line|
puts "\n bumping package: #{line_hash['name']} formula"
begin
bump_pr_response, bump_pr_status = brew_commands.bump_formula_pr(line_hash['name'], line_hash['download_url'], line_hash['checksum'])
bump_pr_response, bump_pr_status = brew_commands.bump_formula_pr(line_hash["name"], line_hash["download_url"], line_hash["checksum"])
puts "#{bump_pr_response}"
rescue
puts "- An error occured whilst bumping package #{line_hash['name']} \n"
puts "- An error occured whilst bumping package #{line_hash["name"]} \n"
return
end
end

View File

@ -16,12 +16,12 @@ class BrewCommands
end
def parse_livecheck_response(livecheck_output)
livecheck_output = livecheck_output.first.gsub(' ', '').split(/:|==>|\n/)
livecheck_output = livecheck_output.first.gsub(" ", "").split(/:|==>|\n/)
# eg: ["burp", "2.2.18", "2.2.18"]
package_name, brew_version, latest_version = livecheck_output
{'name' => package_name, 'current_brew_version' => brew_version, 'livecheck_latest_version' => latest_version}
{"name" => package_name, "current_brew_version" => brew_version, "livecheck_latest_version" => latest_version}
end
def bump_formula_pr(formula_name, url)
@ -48,7 +48,7 @@ class BrewCommands
response = bump_formula_pr(formula_name, download_url)
!response.include? 'Error: These open pull requests may be duplicates'
!response.include? "Error: These open pull requests may be duplicates"
end
end

View File

@ -1,5 +1,5 @@
require 'net/http'
require 'open-uri'
require "net/http"
require "open-uri"
class HomebrewFormula

View File

@ -1,4 +1,4 @@
require_relative 'helpers/api_parser'
require_relative "helpers/api_parser"
api_parser = ApiParser.new