Skip to content

xtask: remove node binary from path#234

Closed
gab8i wants to merge 1 commit intomainfrom
gab_remove_node_in_path_requirement
Closed

xtask: remove node binary from path#234
gab8i wants to merge 1 commit intomainfrom
gab_remove_node_in_path_requirement

Conversation

@gab8i
Copy link
Contributor

@gab8i gab8i commented Feb 15, 2024

Zombienet requires the node binary to be in the path.
Until now, it was sourced before calling 'zombienet.sh',
which made things more complex in CI.
Removing the need to source it before and instead
doing it in the test script makes it easier to work with

Implementing this, I understood better how Duct works, and I had to also change how I was executing the shim. Solving #232 it is probable that the way commands are executed will change again

Copy link
Contributor Author

gab8i commented Feb 15, 2024

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

@gab8i gab8i force-pushed the gab_remove_node_in_path_requirement branch 3 times, most recently from 95b05a3 to 03fead3 Compare February 16, 2024 12:33
@gab8i gab8i force-pushed the gab_remove_node_in_path_requirement branch from 03fead3 to 7bdb25b Compare February 16, 2024 16:15
@gab8i gab8i closed this Feb 18, 2024
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.

1 participant