Skip to content
This repository was archived by the owner on Jan 21, 2022. It is now read-only.

Conversation

@nimakaviani
Copy link
Contributor

@nimakaviani nimakaviani commented Nov 25, 2018

  • document your feature
  • comment your code
  • update the README / how-to-run
  • have tests
  • get reviews

@cfdreddbot
Copy link

✅ Hey nimakaviani! The commit authors and yourself have already signed the CLA.

@nimakaviani nimakaviani force-pushed the enable-fabric branch 2 times, most recently from 44c4d7a to 24ca045 Compare November 27, 2018 03:46
@nimakaviani
Copy link
Contributor Author

checks are passing.

The PR uses Tags in service definitions to determine the type of the service.

@nimakaviani
Copy link
Contributor Author

as part of verifying this PR, @jberkhahn will try and deploy a sample voting application on a CF cluster and verify the functionality of the app when using fabric as the backing blockchain.

nimakaviani and others added 5 commits December 6, 2018 11:07
- read the type of service from the TAGS description of the service
config. `ethereum` and `fabric` are the two types of service known to
the service broker at the moment.
- stop the running broker process after each test
- unifies setup for all the integration tests
- set package dependencies back to the right versions
- log exit code for the deployer when running the pusher fails
- also add log line to capture configuratin json that gets passed to
pusher.js
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants