homebrew/brew
Pull request #24015 quality checks
Code files and functions touched by this pull request are analyzed below.
Quality targets
How does this pull request compare to chosen code quality targets? Read more about code quality targets.
| Target name | Target value | This PR |
|---|---|---|
| Maximum Diff Delta per PR | 500 | 235 |
| Maximum comment count per PR | 10 | 0 |
| Pull request maximum business days between activity | 3 | 1 |
| Pull request maximum business days open | 5 | 1 |
Coupled files
Are there any files from this commit that are always committed alongside another file? This often suggests that there exists code coupling to potentially remove. Read more about code coupling
| Committed together % |
Code file
Coupled code file |
Commits in PR with both |
|---|---|---|
| 77 |
Library/Homebrew/test/bundle/skipper_spec.rb
Library/Homebrew/bundle/skipper.rb |
1 |
| 77 |
Library/Homebrew/bundle/skipper.rb
Library/Homebrew/test/bundle/skipper_spec.rb |
1 |
| 71 |
Library/Homebrew/extend/os/linux/bundle/skipper.rb
Library/Homebrew/test/bundle/skipper_spec.rb |
1 |
| 71 |
Library/Homebrew/extend/os/linux/bundle/skipper.rb
Library/Homebrew/bundle/skipper.rb |
1 |
Biggest files changes in pull request
| Path | View whole file | Lines removed | Lines added | Final line count |
|---|---|---|---|---|
| Library/Homebrew/test/bundle/skipper_spec.rb | BeforeAfter | 21 | 14 | 162 |
| Library/Homebrew/bundle/skipper.rb | BeforeAfter | 1 | 13 | 76 |
| Library/Homebrew/extend/os/linux/bundle/skipper.rb | BeforeAfter | 13 | 3 | 43 |
| Library/Homebrew/extend/os/mac/bundle/skipper.rb | BeforeAfter | 17 | 6 | 23 |
| Library/Homebrew/test/dev-cmd/verify_spec.rb | BeforeAfter | 0 | 10 | 88 |
| Library/Homebrew/test/cask/cask_loader/from_uri_loader_spec.rb | BeforeAfter | 0 | 7 | 64 |
| Library/Homebrew/dev-cmd/verify.rb | BeforeAfter | 0 | 0 | 89 |
| Library/Homebrew/cask/cask_loader.rb | BeforeAfter | 0 | 0 | 934 |
| Library/Homebrew/test/cmd/update_spec.rb | BeforeAfter | 0 | 1 | 531 |
| Library/Homebrew/cmd/update.sh | BeforeAfter | 0 | 0 | --- |
️ Biggest functions grown or shrunk by pull request
Which functions had their line count change as part of this pull request?
| Qualified function name | View whole function | Lines removed | Lines added | Final line count |
|---|---|---|---|---|
| Homebrew::Bundle::Skipper::skip? | BeforeAfter | 1 | 3 | 25 |
| OS::Mac::Bundle::Skipper::ClassMethods::unsupported_reason | BeforeAfter | 0 | 5 | 6 |
| OS::unsupported_reason | BeforeAfter | 1 | 2 | 5 |
| OS::Mac::Bundle::Skipper::ClassMethods::linux_only_entry? | 3 | 0 | --- | |
| OS::Mac::Bundle::Skipper::ClassMethods::skip? | 1 | 0 | --- | |
| OS::Linux::Bundle::Skipper::ClassMethods::requires_wsl? | 1 | 0 | --- | |
| OS::Linux::Bundle::Skipper::ClassMethods::skip? | 1 | 0 | --- |