Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Make network_config_path an Option #5661

Merged
merged 5 commits into from
Apr 17, 2020
Merged

Make network_config_path an Option #5661

merged 5 commits into from
Apr 17, 2020

Conversation

expenses
Copy link
Contributor

Closes #5611.

@expenses expenses requested review from cecton and tomaka April 16, 2020 11:14
Copy link
Contributor

@tomaka tomaka left a comment

Choose a reason for hiding this comment

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

Funny, we don't actually use the parameter in the network code.
But we will need it for #4750

@tomaka tomaka added B0-silent Changes should not be mentioned in any release notes B2-breaksapi labels Apr 16, 2020
@cecton
Copy link
Contributor

cecton commented Apr 16, 2020

Funny, we don't actually use the parameter in the network code.

Ah! Actually that was what I was thinking about. The only occurrence of it I saw was to create the directory. Good to hear that it will finally be used.

cecton
cecton previously requested changes Apr 16, 2020
client/cli/src/commands/mod.rs Outdated Show resolved Hide resolved
@bkchr bkchr merged commit 2d8f3b4 into master Apr 17, 2020
@bkchr bkchr deleted the ashley-network-config-path branch April 17, 2020 12:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Browser node panic on current_dir call
4 participants