Skip to content

(Almost) Passing Felix Tests - Operational Decentralized Distribution#969

Merged
KPrasch merged 24 commits intonucypher:hawksbeardfrom
KPrasch:felitronic
May 15, 2019
Merged

(Almost) Passing Felix Tests - Operational Decentralized Distribution#969
KPrasch merged 24 commits intonucypher:hawksbeardfrom
KPrasch:felitronic

Conversation

@KPrasch
Copy link
Member

@KPrasch KPrasch commented May 14, 2019

This PR goes into EPIC #951

Opens Issues: #976 #975 #974 #973

KPrasch added 24 commits May 14, 2019 15:09
…mtn; Begin the movement to integrate BlockchainPower.
@KPrasch KPrasch requested review from derekpierre, jMyles and tuxxy May 14, 2019 21:20
if self.provider_process:

if not os.path.exists(self.provider_process.data_dir):
os.mkdir(self.provider_process.data_dir)
Copy link
Member

Choose a reason for hiding this comment

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

is a mode needed?

Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm - Yeah explicit is better than implicit?

Copy link
Member

@cygnusv cygnusv left a comment

Choose a reason for hiding this comment

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

Just a few comments. Also, how can I try this?

@KPrasch KPrasch changed the title Passing Felix Tests - Operational Decentralized Distribution (Almost) Passing Felix Tests - Operational Decentralized Distribution May 15, 2019
@KPrasch
Copy link
Member Author

KPrasch commented May 15, 2019

@cygnusv - Im shifting towards getting the test suite to run - Id recommend waiting for another merge or two (into #951) before trying to run this version.

@KPrasch KPrasch merged commit 0cf7bf5 into nucypher:hawksbeard May 15, 2019
@KPrasch KPrasch deleted the felitronic branch July 21, 2020 02:30
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