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.

Sign up for GitHub

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

Jump to bottom

edge case handleing #18621

Closed
wants to merge 2 commits into from
Closed

edge case handleing #18621

wants to merge 2 commits into from

Commits on Oct 24, 2024

  1. Add comprehensive error handling for edge cases during formula builds 

    * **Library/Homebrew/build.rb**
  - Add rescue block to handle errors during the installation of formula
* **Library/Homebrew/cmd/deps.rb**
  - Add rescue blocks to handle errors during dependency processing and dependents processing
* **Library/Homebrew/debrew.rb**
  - Add logging and retry options to the debugging tool
  - Add method to log exception details to a file
* **Library/Homebrew/formula_assertions.rb**
  - Add assert_raises and assert_nothing_raised methods for more exhaustive assertions
* **Library/Homebrew/formula_auditor.rb**
  - Add rescue block to handle errors during formula audits
    @cameronaaron
    cameronaaron committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    7a5f847 View commit details
    Browse the repository at this point in the history

  2. Add edge case handling and improve error messages for dependencies 

    * **dependency_collector.rb**
  - Add edge case handling for unsupported dependency specifications and URL extensions in `parse_spec` method
  - Improve error messages in `fetch` method

* **dependency.rb**
  - Add edge case handling for circular dependencies in `expand` method
  - Improve error messages in `installed?` method

* **formula_installer.rb**
  - Add edge case handling for failed installations in `install` method
  - Improve error messages in `check_install_sanity` method

* **cask_dependent.rb**
  - Add edge case handling for missing dependencies in `initialize` method
  - Improve error messages in `runtime_dependencies` method

* **exceptions.rb**
  - Improve error messages for dependency-related exceptions

* **formula_auditor.rb**
  - Add edge case handling for missing dependencies in `audit_deps` method

* **api.rb**
  - Add edge case handling for missing API responses in `fetch` method
  - Improve error messages in `fetch` method
    @cameronaaron
    cameronaaron committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    39f983e View commit details
    Browse the repository at this point in the history