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

Move a bunch of stuff around#10101

Merged
sorpaas merged 16 commits into
masterfrom
td-refactor
Dec 28, 2018
Merged

Move a bunch of stuff around#10101
sorpaas merged 16 commits into
masterfrom
td-refactor

Conversation

@tomusdrw
Copy link
Copy Markdown
Collaborator

An attempt to clear the directory structure a bit:

  • if crate is specific to one of the client "components" move it inside (e.g move price-info to miner)
  • dissolve tiny utility crates (stop_guard & devtools)
  • create a subdir for account management (Remove/split out account management #9997)
  • unify directory names for crates (use hyphen instead of underscore)
  • rename some crates (e.g. mem -> memzero, additionaly this one could potentially use memsec crate)

@tomusdrw tomusdrw added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Dec 24, 2018
@NikVolf NikVolf added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Dec 24, 2018
@ordian
Copy link
Copy Markdown
Member

ordian commented Dec 27, 2018

@sorpaas sorpaas merged commit ff0095a into master Dec 28, 2018
@sorpaas sorpaas deleted the td-refactor branch December 28, 2018 09:33
@5chdn 5chdn added this to the 2.3 milestone Dec 28, 2018
5chdn pushed a commit that referenced this pull request Jan 2, 2019
Due to #10101 new directory structure, some links needed to be fixed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants