Skip to content

Commit

Permalink
Make brew typecheck --update --suggest-typed bump strictness further
Browse files Browse the repository at this point in the history
- Not only `false` to `true` but `true` to `strict`.
- Only humans every run this, but our goal is to increase the
  typechecking in our files to get to `strict` everywhere so
  let's make that easy to remember to do.
  • Loading branch information
issyl0 committed Jun 2, 2024
1 parent 11f042e commit 80be27d
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions Library/Homebrew/dev-cmd/typecheck.rb
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,12 @@ def run
safe_system "bundle", "exec", "parlour"

if args.suggest_typed?
ohai "Bumping Sorbet `typed` sigils..."
ohai "Checking if we can bump Sorbet `typed` sigils..."
# --sorbet needed because of https://github.com/Shopify/spoom/issues/488
safe_system "bundle", "exec", "spoom", "srb", "bump", "--dry", "--sorbet",
"#{Gem.bin_path("sorbet", "srb")} tc"
safe_system "bundle", "exec", "spoom", "srb", "bump", "--dry", "--from", "false", "--to", "true",

Check warning on line 63 in Library/Homebrew/dev-cmd/typecheck.rb

View check run for this annotation

Codecov / codecov/patch

Library/Homebrew/dev-cmd/typecheck.rb#L63

Added line #L63 was not covered by tests
"--sorbet", "#{Gem.bin_path("sorbet", "srb")} tc"
safe_system "bundle", "exec", "spoom", "srb", "bump", "--dry", "--from", "true", "--to", "strict",

Check warning on line 65 in Library/Homebrew/dev-cmd/typecheck.rb

View check run for this annotation

Codecov / codecov/patch

Library/Homebrew/dev-cmd/typecheck.rb#L65

Added line #L65 was not covered by tests
"--sorbet", "#{Gem.bin_path("sorbet", "srb")} tc"
end

return
Expand Down

0 comments on commit 80be27d

Please sign in to comment.