Skip to content

Add config path to acceptance tests#946

Merged
yaahc merged 3 commits intoZcashFoundation:mainfrom
oxarbitrage:issue943
Sep 3, 2020
Merged

Add config path to acceptance tests#946
yaahc merged 3 commits intoZcashFoundation:mainfrom
oxarbitrage:issue943

Conversation

@oxarbitrage
Copy link
Contributor

@oxarbitrage oxarbitrage commented Aug 24, 2020

Fixes #943 in the first commit and resolves #976 in the second.

yaahc
yaahc previously approved these changes Aug 24, 2020
Copy link
Contributor

@yaahc yaahc left a comment

Choose a reason for hiding this comment

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

looks great ^_^

@teor2345 teor2345 added A-rust Area: Updates to Rust code C-bug Category: This is a bug E-easy S-blocked Status: Blocked on other tasks labels Aug 25, 2020
Copy link
Contributor

@teor2345 teor2345 left a comment

Choose a reason for hiding this comment

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

This PR conflicts with PR #939.

I'd like to merge PR #939 first, and then rewrite this PR to use the new ConfigMode enum:
https://github.com/ZcashFoundation/zebra/pull/939/files#diff-33c347fc0c4532e93ecd68f7cef1fb07R16

@teor2345 teor2345 removed the S-blocked Status: Blocked on other tasks label Aug 31, 2020
@teor2345
Copy link
Contributor

I just merged PR #939, please feel free to rebase this PR when you get time.

Copy link
Contributor

@yaahc yaahc left a comment

Choose a reason for hiding this comment

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

@yaahc yaahc dismissed teor2345’s stale review September 3, 2020 20:12

previous comments have been addressed.

@yaahc yaahc merged commit 5485f44 into ZcashFoundation:main Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-rust Area: Updates to Rust code C-bug Category: This is a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

remove option to read config from current directory Set the config path in the zebrad acceptance tests

3 participants