- Previously I thought that comments were fine to discourage people from wasting their time trying to bump things that used `undef` that Sorbet didn't support. But RuboCop is better at this since it'll complain if the comments are unnecessary. - Suggested in https://github.com/Homebrew/brew/pull/18018#issuecomment-2283369501. - I've gone for a mixture of `rubocop:disable` for the files that can't be `typed: strict` (use of undef, required before everything else, etc) and `rubocop:todo` for everything else that should be tried to make strictly typed. There's no functional difference between the two as `rubocop:todo` is `rubocop:disable` with a different name. - And I entirely disabled the cop for the docs/ directory since `typed: strict` isn't going to gain us anything for some Markdown linting config files. - This means that now it's easier to track what needs to be done rather than relying on checklists of files in our big Sorbet issue: ```shell $ git grep 'typed: true # rubocop:todo Sorbet/StrictSigil' | wc -l 268 ``` - And this is confirmed working for new files: ```shell $ git status On branch use-rubocop-for-sorbet-strict-sigils Untracked files: (use "git add <file>..." to include in what will be committed) Library/Homebrew/bad.rb Library/Homebrew/good.rb nothing added to commit but untracked files present (use "git add" to track) $ brew style Offenses: bad.rb:1:1: C: Sorbet/StrictSigil: Sorbet sigil should be at least strict got true. ^^^^^^^^^^^^^ 1340 files inspected, 1 offense detected ```
		
			
				
	
	
		
			46 lines
		
	
	
		
			870 B
		
	
	
	
		
			Ruby
		
	
	
	
	
	
			
		
		
	
	
			46 lines
		
	
	
		
			870 B
		
	
	
	
		
			Ruby
		
	
	
	
	
	
| # typed: true # rubocop:todo Sorbet/StrictSigil
 | |
| # frozen_string_literal: true
 | |
| 
 | |
| require "keg"
 | |
| 
 | |
| # Helper functions for pinning a formula.
 | |
| class FormulaPin
 | |
|   def initialize(formula)
 | |
|     @formula = formula
 | |
|   end
 | |
| 
 | |
|   def path
 | |
|     HOMEBREW_PINNED_KEGS/@formula.name
 | |
|   end
 | |
| 
 | |
|   def pin_at(version)
 | |
|     HOMEBREW_PINNED_KEGS.mkpath
 | |
|     version_path = @formula.rack/version
 | |
|     path.make_relative_symlink(version_path) if !pinned? && version_path.exist?
 | |
|   end
 | |
| 
 | |
|   def pin
 | |
|     latest_keg = @formula.installed_kegs.max_by(&:scheme_and_version)
 | |
|     return if latest_keg.nil?
 | |
| 
 | |
|     pin_at(latest_keg.version)
 | |
|   end
 | |
| 
 | |
|   def unpin
 | |
|     path.unlink if pinned?
 | |
|     HOMEBREW_PINNED_KEGS.rmdir_if_possible
 | |
|   end
 | |
| 
 | |
|   def pinned?
 | |
|     path.symlink?
 | |
|   end
 | |
| 
 | |
|   def pinnable?
 | |
|     !@formula.installed_prefixes.empty?
 | |
|   end
 | |
| 
 | |
|   def pinned_version
 | |
|     Keg.new(path.resolved_path).version if pinned?
 | |
|   end
 | |
| end
 |