Skip to content
New issue

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

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

Already on GitHub? # to your account

Make build_test_hooks.sh compatible with macOS #440

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

tequdev
Copy link
Collaborator

@tequdev tequdev commented Feb 13, 2025

High Level Overview of Change

  • Ensure build_test_hooks.sh works on macOS environments
  • Maintain cross-platform compatibility between Linux and macOS

Note: requires 'ggrep' to be installed via Homebrew on macOS

Context of Change

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Tests (you added tests for code that already exists, or your new feature included in this PR)
  • Documentation update
  • Chore (no impact to binary, e.g. .gitignore, formatting, dropping support for older tooling)
  • Release

API Impact

  • Public API: New feature (new methods and/or new fields)
  • Public API: Breaking change (in general, breaking changes should only impact the next api_version)
  • libxrpl change (any change that may affect libxrpl or dependents of libxrpl)
  • Peer protocol change (must be backward compatible or bump the peer protocol version)

- Ensure build_test_hooks.sh works on macOS environments
- Maintain cross-platform compatibility between Linux and macOS

Note: requires 'ggrep' to be installed via Homebrew on macOS
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant