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

serde is no longer util dependency #1534

Merged
merged 26 commits into from Jul 6, 2016
Merged

serde is no longer util dependency #1534

merged 26 commits into from Jul 6, 2016

Conversation

debris
Copy link
Collaborator

@debris debris commented Jul 4, 2016 •
edited

based on #1474

changes:

  • serde is no longer util dependency
  • updated hyper version used by igd, now it's the same version as the one used by rpc
  • hash / uint serialization is moved to rpc module
  • Table and UsingQueue are now standalone libraries (but they remained a part of util)
  • reduced util compilation time 283.07s -> 250.67s (including all dependencies)

debris added 24 commits June 28, 2016 15:26
@debris
removed old json-tests
2a39c06
@debris
simplify folds in triehash.rs
1878d38
@debris
removed unused json_aid
6a7f619
@debris
removed unused squeeze.rs
1880b11
@debris
Merge branch 'master' into util_cleanup
77299a4
@debris
json branching tests for trie
321b16c
@debris
removing todos from util
c0dc35c
@debris
separated UsingQueue and Table
778efa6
@debris
further cleanup, removing unused code
187865a
@debris
serde serialization of hash moved to rpc module
23d015b
@debris
Merge branch 'master' of github.com:ethcore/parity into removing_todos
d016a21
@debris
uint wrapper for rpc in progress
896e5fd
@debris
serialization of uint moved to rpc module
cd73b51
@debris
Merge branch 'master' of github.com:ethcore/parity into removing_todos
fd1f717
@debris
Merge branch 'master' of github.com:ethcore/parity into util_cleanup
c39b8bb
@debris
Merge branch 'util_cleanup' into removing_todos
bbf117d
@debris
updated eth-secp256k1
3c67bd7
@debris
Merge branch 'master' of github.com:ethcore/parity into removing_todos
430d9d1
@debris
updated igd, serde is no longer dependency of util
021735d
@debris
Merge branch 'master' of github.com:ethcore/parity into util_cleanup
c9fd532
@debris
loading trie consensus tests
d8b2844
@debris
Merge branch 'master' of github.com:ethcore/parity into util_cleanup
a26133c
@debris
Merge branch 'util_cleanup' into removing_todos
46100c7
@debris
renamed aliases in rpc imports
647d4e3
@debris debris added the A0-pleasereview 🤓 Pull request needs code review. label Jul 4, 2016
@arkpar
Copy link
Collaborator

arkpar commented Jul 5, 2016

Too large to review in detail. Looks good in general

@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jul 5, 2016
@NikVolf
Copy link
Contributor

NikVolf commented Jul 5, 2016

based on #1474
@gavofyork can you re-review that?

@gavofyork gavofyork added A7-looksgoodcantmerge 🙄 Pull request is reviewed well, but cannot be merged due to conflicts. and removed A8-looksgood 🦄 Pull request is reviewed well. labels Jul 5, 2016
@gavofyork gavofyork merged commit bcb63bc into master Jul 6, 2016
@gavofyork gavofyork deleted the removing_todos branch July 6, 2016 09:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
A7-looksgoodcantmerge 🙄 Pull request is reviewed well, but cannot be merged due to conflicts.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@debris @arkpar @NikVolf @gavofyork