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

Use associated type M::Error instead of Error #8308

Merged
merged 1 commit into from
Apr 5, 2018
Merged

Use associated type M::Error instead of Error #8308

merged 1 commit into from
Apr 5, 2018

Conversation

kseo
Copy link
Contributor

@kseo kseo commented Apr 4, 2018

No description provided.

@debris
Copy link
Collaborator

debris commented Apr 4, 2018

[clabot:check]

@debris debris added A4-clasignoffneeded 📛 Pull request requires author to sign off on CLA before review/merge can begin. M4-core ⛓ Core client code / Rust. labels Apr 4, 2018
@debris
Copy link
Collaborator

debris commented Apr 4, 2018

let's wait for clabot to wake up

@5chdn 5chdn added this to the 1.11 milestone Apr 5, 2018
@ddorgan
Copy link
Collaborator

ddorgan commented Apr 5, 2018

[clabot:check]

@parity-cla-bot
Copy link

It looks like @kseo hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement at the following URL: https://cla.parity.io

Once you've signed, plesae reply to this thread with [clabot:check] to prove it.

Many thanks,

Parity Technologies CLA Bot

@kseo
Copy link
Contributor Author

kseo commented Apr 5, 2018

[clabot:check]

@parity-cla-bot
Copy link

It looks like @kseo signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@niklasad1 niklasad1 added A8-looksgood 🦄 Pull request is reviewed well. and removed A4-clasignoffneeded 📛 Pull request requires author to sign off on CLA before review/merge can begin. labels Apr 5, 2018
@niklasad1 niklasad1 merged commit 0d2993e into openethereum:master Apr 5, 2018
@kseo kseo deleted the error branch April 5, 2018 09:34
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.

6 participants