Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix HOMEBREW_*BREW_WRAPPER behaviour #19054

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Conversation

MikeMcQuaid
Copy link
Member

@MikeMcQuaid MikeMcQuaid commented Jan 8, 2025

  • handle reading from variables that may not be defined yet
  • fix incorrect env_config.rbi file entry
  • avoid unnecessary handling of defined variables
  • add HOMEBREW_NO_FORCE_BREW_WRAPPER to disable HOMEBREW_FORCE_BREW_WRAPPER functionality if needed

@MikeMcQuaid MikeMcQuaid requested review from Bo98 and carlocab January 8, 2025 14:05
- handle reading from variables that may not be defined yet
- avoid unnecessary handling of defined variables
- fix incorrect `env_config.rbi` file entry
- add `HOMEBREW_NO_FORCE_BREW_WRAPPER` to disable
  `HOMEBREW_FORCE_BREW_WRAPPER` functionality if needed
@MikeMcQuaid MikeMcQuaid enabled auto-merge January 8, 2025 14:11
@MikeMcQuaid MikeMcQuaid added this pull request to the merge queue Jan 8, 2025
auto-merge was automatically disabled January 8, 2025 14:32

Pull Request is not mergeable

auto-merge was automatically disabled January 8, 2025 15:02

Pull Request is not mergeable

Merged via the queue into master with commit 4f0f591 Jan 8, 2025
28 checks passed
@MikeMcQuaid MikeMcQuaid deleted the brew_wrapper_fixes branch January 8, 2025 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants