Skip to content

Conversation

ofalvai
Copy link
Collaborator

@ofalvai ofalvai commented Aug 12, 2025

Checklist

Version

Requires a MAJOR/MINOR/PATCH version update

Context

Right now, integration tests create a new sandboxed asdf environment for each test (in order to make it not depend on the system-wide asdf installation and config).

We could flip this and use the test cases for testing the system-wide asdf installation and shell config.

The toolprovider thing relies on a preinstalled asdf version (right now), as well as preinstalled asdf plugin versions. If we accidentally change the versions of asdf plugins (which are impossible to pin), it could break the entire feature.

Changes

Test builds:

Investigation details

Decisions

@ofalvai ofalvai force-pushed the push-tnslkqqwqvtn branch 6 times, most recently from 5803bf8 to a07dec1 Compare August 12, 2025 16:34
@ofalvai ofalvai force-pushed the push-tnslkqqwqvtn branch 2 times, most recently from bf99efc to 50a56ca Compare August 13, 2025 11:58
@ofalvai ofalvai marked this pull request as ready for review August 13, 2025 12:03
pgyula
pgyula previously approved these changes Aug 13, 2025
@ofalvai ofalvai force-pushed the push-tnslkqqwqvtn branch from 97f6c69 to 224928e Compare August 13, 2025 14:21
@ofalvai ofalvai merged commit ecb045c into master Aug 13, 2025
8 checks passed
@ofalvai ofalvai deleted the push-tnslkqqwqvtn branch August 13, 2025 14:33
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.

3 participants