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

[testing] Support direnv to simplify usage of test tooling #3651

Merged
merged 5 commits into from
Jan 27, 2025

Conversation

marun
Copy link
Contributor

@marun marun commented Jan 16, 2025

Why this should be merged

Adds a .envrc file for use with direnv enables automatic configuration of paths and environment variables to simplify usage of test tooling. For example, ginkgo and tmpnetctl can be invoked directly and without having to specify the --avalanchego-path or --plugin-dir flags. It also avoids the use of global paths for those of us that develop concurrently on multiple local worktrees.

How this works

  • Add a .envrc file to support automatic shell configuration for those who have direnv installed
  • Ensures backwards-compatible behavior for those who don't know about or don't want to use direnv

How this was tested

  • CI to avoid regression
  • Manually for direnv usage

Need to be documented in RELEASES.md?

N/A

@marun marun added the testing This primarily focuses on testing label Jan 16, 2025
@marun marun self-assigned this Jan 16, 2025
@maru-ava maru-ava assigned maru-ava and unassigned marun Jan 22, 2025
@StephenButtolph StephenButtolph added this pull request to the merge queue Jan 27, 2025
Merged via the queue into master with commit ab46198 Jan 27, 2025
22 checks passed
@StephenButtolph StephenButtolph deleted the direnv branch January 27, 2025 20:48
tsachiherman pushed a commit that referenced this pull request Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing This primarily focuses on testing
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants