Skip to content
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.

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

Remove mypy errors around SendSecretReveal #3219

Merged
merged 2 commits into from
Jan 9, 2019

Conversation

pirapira
Copy link
Contributor

@pirapira pirapira commented Jan 3, 2019

In particular, this commit removes the "unknown name" error in
mediated_transfer/state.py about SendSecretReveal. After this, it
becomes possible to remove warnings from raiden/docs compilation.

@pirapira pirapira force-pushed the SendSecretReveal branch 2 times, most recently from ae2c582 to 6d440f7 Compare January 3, 2019 13:21
raiden/transfer/events.py Outdated Show resolved Hide resolved
@pirapira
Copy link
Contributor Author

pirapira commented Jan 7, 2019

And why didn't I submit the change in Makefile?

In particular, this commit removes the "unknown name" error in
mediated_transfer/state.py about SendSecretReveal. After this, it
becomes possible to remove warnings from raiden/docs compilation.
@pirapira
Copy link
Contributor Author

pirapira commented Jan 7, 2019

I added the Makefile change, too.

@pirapira
Copy link
Contributor Author

pirapira commented Jan 7, 2019

@LefterisJP @andrevmatos it should be better now.

@pirapira
Copy link
Contributor Author

pirapira commented Jan 9, 2019

Ping @andrevmatos

Copy link
Contributor

@andrevmatos andrevmatos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@pirapira pirapira merged commit 89896c3 into raiden-network:master Jan 9, 2019
@pirapira pirapira deleted the SendSecretReveal branch January 9, 2019 13:40
@LefterisJP LefterisJP mentioned this pull request Feb 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants