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

CI/CD sync #340

Merged
merged 14 commits into from
May 7, 2020
Merged

CI/CD sync #340

merged 14 commits into from
May 7, 2020

Conversation

tonymorony
Copy link

No description provided.

tonymorony and others added 9 commits April 8, 2020 09:15
Mihailo: `changed vout order so it is comparing marker value instead of vout to destination value.`
fix chanelsCC test vouts
fix broken secret reveal test
add rpc response validation
refund check no longer depends on the vout number
prevents test failure on different branches due to ChannelsCC changes
In some cases CCs throwing exception such as:

`{'error': {'code': -3, 'message': 'Amount out of range'}`

And in some cases errors responses forming manually, e.g.:

`{'error': {'code': -3, 'message': 'Amount out of range'}`

lets handle both options (thanks to @dimxy for advise)
@tonymorony tonymorony requested a review from ca333 April 29, 2020 12:52
@tonymorony tonymorony removed the request for review from ca333 April 29, 2020 13:55
@tonymorony
Copy link
Author

new patch(es) needed for all the latest CC updates we've merged from jl777/dev - will work on that this week

@ca333 ca333 merged commit f73e51e into research May 7, 2020
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