Home » Oreps » Brew » Pull Requests
homebrew/brew
🍺 The Package Manager for Everywhere
Watch
1
Commit Activity
Pull Requests
Directory Browser
Velocity History
Releases

Pull Requests

Review 580 pull requests

Merged
Prefill `app` and `zap` in `brew create --cask`
 
 0.2 days Authoring commits before pull request opened for review
 
 0.2 days Awaiting first review
 
 0.6 days Revising work while processing 2 reviewers who left 13 comments

Total time for pull request 1.0 business day from first commit authored to merge

2 reviewers
awaited review for 1.5 business hours
#24316 with 1 commit opened 21 days ago by bevanjkay
brew_create_zap
Open October 21 - October 22
Review Code View on GitHub
Total Diff Delta
276
Before PR submitted
276
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
276
Diff Delta
224 under target
Total comments
13
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
13
comments
3 over target
Merged
Require `zap` stanzas in new casks
 
 0.3 days Authoring commits before pull request opened for review
 
 0.0 days Awaiting first review
 
 1.3 days Revising work while processing 2 reviewers who left 14 comments

Total time for pull request 1.6 business days from first commit authored to merge

2 reviewers
awaited review for 0.1 business hours
#24315 with 1 commit opened 22 days ago by bevanjkay
zap_audit
Open October 21 - October 22
Review Code View on GitHub
Total Diff Delta
269
Before PR submitted
269
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
269
Diff Delta
231 under target
Total comments
14
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
14
comments
4 over target
Merged
Don't flag newer CLT releases as outdated
 
 0.0 days Authoring commits before pull request opened for review
 
 0.0 days Awaiting first review

Total time for pull request 0.0 business days from first commit authored to merge

2 reviewers
awaited review for 0.0 business hours
#24314 with 1 commit opened 22 days ago by p-linnane
clt-newer-release-guard
Opened and closed October 21
Review Code View on GitHub
Total Diff Delta
27
Before PR submitted
27
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
27
Diff Delta
473 under target
Total comments
2
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
2
comments
8 under target
Closed
Fix ZIP volume label specs on macOS 27
 
 0 days Authoring commits before pull request opened for review
 
 0 days Awaiting first review

Total time for pull request 0 business days from first commit to closed PR

2 reviewers
awaited review for 0.0 business hours
#24313 with 1 commit opened 22 days ago by p-linnane
zip-volume-label-specs
Opened and closed October 21
Review Code View on GitHub
Total Diff Delta
50
Before PR submitted
50
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
50
Diff Delta
450 under target
Total comments
2
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
2
comments
8 under target
Closed
generate-cask-ci-matrix: skip signing on macOS 15
 
 0 days Authoring commits before pull request opened for review
 
 0 days Awaiting first review
 
 1 day Revising work while processing 3 reviewers who left 2 comments

Total time for pull request 1 business day from first commit to closed PR

3 reviewers
awaited review for 0.0 business hours
#24312 with 1 commit opened 22 days ago by bevanjkay
handoff_brew_skip_audit
Opened and closed October 21
Review Code View on GitHub
Total Diff Delta
88
Before PR submitted
88
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
88
Diff Delta
412 under target
Total comments
2
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
2
comments
8 under target
Merged
cask/audit: reject new casks requiring Rosetta
 
 0.0 days Authoring commits before pull request opened for review
 
 0.0 days Awaiting first review

Total time for pull request 0.0 business days from first commit authored to merge

2 reviewers
awaited review for 0.0 business hours
#24309 with 1 commit opened 22 days ago by bevanjkay
casks_audit_reject_rosetta
Opened and closed October 20
Review Code View on GitHub
Total Diff Delta
81
Before PR submitted
81
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
81
Diff Delta
419 under target
Total comments
5
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
5
comments
5 under target
Merged
Keep patches in stable blocks during bumps
N/A 0.2 days Issue creation to work beginning
 
 0.1 days Authoring commits before pull request opened for review
 
 0.1 days Awaiting first review

Total time for pull request 0.2 business days from first commit authored to merge

2 reviewers
awaited review for 0.8 business hours
#24307 with 1 commit opened 22 days ago by MikeMcQuaid
autobump-stable-patch-removal
Opened and closed October 20
Review Code View on GitHub
Autobump patch removal from stable block leaves the formula in a broken style
Total Diff Delta
13
Before PR submitted
13
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
13
Diff Delta
487 under target
Total comments
1
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
1
comment
9 under target
Merged
Treat archive filenames literally
 
 0.0 days Authoring commits before pull request opened for review
 
 0.0 days Awaiting first review

Total time for pull request 0.0 business days from first commit authored to merge

2 reviewers
awaited review for 0.0 business hours
#24305 with 1 commit opened 23 days ago by p-linnane
zip-literal-archive-paths
Opened and closed October 20
Review Code View on GitHub
Total Diff Delta
155
Before PR submitted
155
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
155
Diff Delta
345 under target
Total comments
2
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
2
comments
8 under target
Merged
Extract tarballs with a UTF-8 locale
 
 0.0 days Authoring commits before pull request opened for review
 
 0.0 days Awaiting first review

Total time for pull request 0.0 business days from first commit authored to merge

2 reviewers
awaited review for 0.0 business hours
#24304 with 1 commit opened 23 days ago by IngmarStein
tar-utf8-locale
Opened and closed October 20
Review Code View on GitHub
Total Diff Delta
20
Before PR submitted
20
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
20
Diff Delta
480 under target
Total comments
5
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
5
comments
5 under target
Merged
Match newlines in tap path regex
 
 0.0 days Authoring commits before pull request opened for review
 
 0.0 days Awaiting first review

Total time for pull request 0.0 business days from first commit authored to merge

2 reviewers
awaited review for 0.0 business hours
#24303 with 1 commit opened 23 days ago by p-linnane
tap-trust-newline
Opened and closed October 20
Review Code View on GitHub
Total Diff Delta
52
Before PR submitted
52
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
52
Diff Delta
448 under target
Total comments
2
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
2
comments
8 under target
Merged
Combine dependency role tags from API metadata
 
 0.8 days Authoring commits before pull request opened for review
 
 0.2 days Awaiting first review

Total time for pull request 1.0 business day from first commit authored to merge

2 reviewers
awaited review for 1.5 business hours
#24302 with 1 commit opened 23 days ago by p-linnane
api-dependency-tag-union
Opened and closed October 20
Review Code View on GitHub
Total Diff Delta
155
Post-merge
77
Before PR submitted
77
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
155
Diff Delta
345 under target
Total comments
2
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
2
comments
8 under target
Merged
Warn when the upstream remote isn't `origin`
 
 0.0 days Authoring commits before pull request opened for review
 
 0.0 days Awaiting first review
 
 0.3 days Revising work while processing 2 reviewers who left 15 comments

Total time for pull request 0.3 business days from first commit authored to merge

2 reviewers
awaited review for 0.1 business hours
#24299 with 2 commits opened 23 days ago by HaraldNordgren
changed-files-upstream-remote
Open October 19 - October 20
Review Code View on GitHub
Total Diff Delta
0
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
0
Diff Delta
500 under target
Total comments
15
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
15
comments
5 over target
Merged
cask/audit: skip livecheck requirement for :latest
 
 0.0 days Authoring commits before pull request opened for review
 
 0.3 days Awaiting first review

Total time for pull request 0.3 business days from first commit authored to merge

2 reviewers
awaited review for 2.4 business hours
#24297 with 1 commit opened 23 days ago by bevanjkay
fix_livecheck
Opened and closed October 19
Review Code View on GitHub
Total Diff Delta
50
Post-merge
25
Before PR submitted
25
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
50
Diff Delta
450 under target
Total comments
2
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
2
comments
8 under target
Merged
advisory-match: exclude reviewed false aliases
 
 0.0 days Authoring commits before pull request opened for review
 
 0.0 days Awaiting first review

Total time for pull request 0.0 business days from first commit authored to merge

2 reviewers
awaited review for 0.0 business hours
#24296 with 1 commit opened 24 days ago by p-linnane
advisory-match-alias-exclusions
Opened and closed October 19
Review Code View on GitHub
Total Diff Delta
214
Before PR submitted
214
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
214
Diff Delta
286 under target
Total comments
1
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
1
comment
9 under target
Closed
Preserve cask backups across lifecycle phases
N/A 0 days Issue creation to work beginning
 
 0 days Authoring commits before pull request opened for review
 
 0 days Awaiting first review

Total time for pull request 0 business days from first commit to closed PR

3 reviewers
awaited review for 0.0 business hours
#24295 with 1 commit opened 24 days ago by MikeMcQuaid
preserve-cask-lifecycle-temp
Open October 18 - October 19
Review Code View on GitHub
Cask lifecycle backups in {{temp}} are deleted by sandbox cleanup before restoration
Total Diff Delta
244
Before PR submitted
244
Target
500
Team average
165
Team average reflects all PRs in the brew repo over the past 30 days.
244
Diff Delta
256 under target
Total comments
1
Target
10
Team average
3
Team average reflects all PRs in the brew repo over the past 30 days.
1
comment
9 under target
GitClear uses cookies to ensure you get the best experience on our website. Learn more about our cookie policy