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

Merge parity/dir.rs and ethstore/src/dir/paths.rs #4972

Closed
tomusdrw opened this issue Mar 21, 2017 · 4 comments
Closed

Merge parity/dir.rs and ethstore/src/dir/paths.rs #4972

tomusdrw opened this issue Mar 21, 2017 · 4 comments
Labels
F6-refactor 📚 Code needs refactoring. M4-core ⛓ Core client code / Rust. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow.

Comments

@tomusdrw
Copy link
Collaborator

No description provided.

@tomusdrw tomusdrw added F6-refactor 📚 Code needs refactoring. M4-core ⛓ Core client code / Rust. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow. labels Mar 21, 2017
@tomusdrw tomusdrw mentioned this issue Mar 21, 2017
Merged
@guanqun
Copy link
Contributor

guanqun commented May 10, 2017 •
edited
Loading

Is this still a valid issue? If so, I'll can help fix this, this seems to be a small ticket. Basically I believe we'd move stuff move ethstore/src/dir/paths.rs to util/path/src/lib.rs and unify some of these functions.

@debris
Copy link
Collaborator

debris commented May 12, 2017

Yes it is. Any help is much appreciated ;)

@guanqun
Copy link
Contributor

guanqun commented May 12, 2017

Thank you @debris I'll get some spare time to fix this.

@debris debris changed the title Merge util/path/src/lib.rs and ethstore/src/dir/paths.rs Merge parity/dir.rs and ethstore/src/dir/paths.rs Aug 1, 2017
@debris
Copy link
Collaborator

debris commented Aug 1, 2017

closing in favor of #6209

@debris debris closed this as completed Aug 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
F6-refactor 📚 Code needs refactoring. M4-core ⛓ Core client code / Rust. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@guanqun @tomusdrw @debris