Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Following on #347 (comment), this makes the
simple
example again test against the checkout, instead of what's in the lockfile. This benefits the direnv test, which can now pick up on any breakages.I thought this also affected other examples, but noticed they all have a
devenv.yaml
path override. At the same time, there's a todo to expanddevenv init
to other examples, so these would also be affected down the line.This PR removes all those path overrides, and instead adds them in CI right before running the test.
This modification makes the lockfiles a little useless, so I've removed them. That also saves a few kb in releases, and helps us catch any breakages via nixpkgs or other inputs.I also noticed the comment in
examples/simple/devenv.yaml
was leaking indevenv init
. This can be seen in the 0.5.1 release. With this change, we can get rid of it.