test: build backend missing executable#3527
Merged
Hofer-Julian merged 4 commits intoprefix-dev:mainfrom Apr 11, 2025
Merged
Conversation
Contributor
|
Can you update the lock file @baszalmstra? |
Hofer-Julian
reviewed
Apr 7, 2025
pixi.toml
Outdated
Comment on lines
+197
to
+198
| [environments.recipes] | ||
| features = ["recipes"] |
Contributor
There was a problem hiding this comment.
Maybe add an inline table a few lines later like we do for the other environments?
Contributor
Author
There was a problem hiding this comment.
I added it like this because I like that everything related to that environment is contained together instead of being spread out over the file. Would be happy to revert if you think its unclearer.
Contributor
There was a problem hiding this comment.
I don't mind in general, but I prefer to keep it consistent. As is we define all environments together at the end.
Contributor
Author
There was a problem hiding this comment.
I moved the definition to the environments table.
Hofer-Julian
approved these changes
Apr 11, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Adds a test to verify the error that occurs when a build backend is used that does not expose the expected executable.