Skip to content

Direnv (.envrc) adds ./bin to path#4810

Open
Gerg wants to merge 1 commit intomainfrom
direnv_add_bin_to_path
Open

Direnv (.envrc) adds ./bin to path#4810
Gerg wants to merge 1 commit intomainfrom
direnv_add_bin_to_path

Conversation

@Gerg
Copy link
Member

@Gerg Gerg commented Jan 27, 2026

  • So that the spring-ified rspec binary (and other spring binstubs) run
    by default
  • This makes tests run faster by default

Continuation of #4802


Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

  • A short explanation of the proposed change:

  • An explanation of the use cases your change solves

  • Links to any other associated PRs

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@Gerg Gerg force-pushed the direnv_add_bin_to_path branch from d6c69db to b0da8bc Compare January 27, 2026 08:04
@Gerg Gerg changed the title Direnv add bin to path Direnv (.envrc) adds ./bin to path Jan 27, 2026
@Gerg Gerg marked this pull request as ready for review February 11, 2026 07:26
@Gerg Gerg mentioned this pull request Mar 3, 2026
5 tasks
@Gerg Gerg force-pushed the direnv_add_bin_to_path branch from b0da8bc to 4b251ad Compare March 3, 2026 16:39
johha
johha previously approved these changes Mar 4, 2026
Copy link
Contributor

@johha johha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

- So that the spring-ified rspec binary (and other spring binstubs) run
  by default
- This makes tests run faster by default
@Gerg Gerg force-pushed the direnv_add_bin_to_path branch from 4b251ad to c583ab9 Compare March 6, 2026 18:27
@Gerg Gerg requested a review from johha March 6, 2026 18:28
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.

2 participants