New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fully qualify std::result::Result in derives #42

Merged

MaksymZavershynskyi merged 1 commit into near:master from nhynes:qualify-paths

Nov 4, 2019

Contributor

nhynes commented Nov 4, 2019

This allows borsh to be used with custom Result types like anyhow::Result.


          Fully qualify std::result::Result in derives

20d82cd

nhynes requested a review from MaksymZavershynskyi as a code owner

November 4, 2019 22:31

MaksymZavershynskyi approved these changes

View reviewed changes

MaksymZavershynskyi merged commit d5f02c0 into near:master

Contributor Author

nhynes commented Nov 4, 2019

Well, this is embarrassing (my first rg missed a few instances). I'll not create another PR in case you want to roll this into a later commit.

MaksymZavershynskyi mentioned this pull request

Use fully-qualified names for types in macroses #43

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment