Skip to content

Commit

Permalink
fix: use backticks in audit
Browse files Browse the repository at this point in the history
Co-authored-by: Mike McQuaid <[email protected]>
  • Loading branch information
SMillerDev and MikeMcQuaid committed Jan 16, 2024
1 parent b308bf5 commit dccfc68
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions Library/Homebrew/formula_auditor.rb
Original file line number Diff line number Diff line change
Expand Up @@ -596,20 +596,20 @@ def audit_wayback_url

regex = %r{^https?://web\.archive\.org}

Check warning on line 597 in Library/Homebrew/formula_auditor.rb

View check run for this annotation

Codecov / codecov/patch

Library/Homebrew/formula_auditor.rb#L597

Added line #L597 was not covered by tests
if regex.match?(formula.stable.url)
problem "Formula with a Wayback URL should be deprecated with `:repo_removed`"
problem "Formula with a Wayback Machine `url` should be deprecated with `:repo_removed`"
return

Check warning on line 600 in Library/Homebrew/formula_auditor.rb

View check run for this annotation

Codecov / codecov/patch

Library/Homebrew/formula_auditor.rb#L600

Added line #L600 was not covered by tests
end

if regex.match?(formula.homepage)
problem "Formula with a Wayback homepage should find an alternative homepage"
problem "Formula with a Wayback Machine `homepage` should find an alternative `homepage`"
return

Check warning on line 605 in Library/Homebrew/formula_auditor.rb

View check run for this annotation

Codecov / codecov/patch

Library/Homebrew/formula_auditor.rb#L605

Added line #L605 was not covered by tests
end

return unless formula.head

return unless regex.match?(formula.head.url)

problem "Formula with a Wayback `HEAD` URL should not have a `HEAD` definition"
problem "Formula with a Wayback `head` should not have a `head` definition"

Check warning on line 612 in Library/Homebrew/formula_auditor.rb

View check run for this annotation

Codecov / codecov/patch

Library/Homebrew/formula_auditor.rb#L612

Added line #L612 was not covered by tests
end

def audit_github_repository_archived
Expand Down

0 comments on commit dccfc68

Please sign in to comment.