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

Add marker module #9

Merged
merged 24 commits into from
Jan 25, 2021
Merged

Add marker module #9

merged 24 commits into from
Jan 25, 2021

Conversation

iramiller
Copy link
Member

Initial import of the marker module from upstream private Provenance blockchain. This version includes some basic cli based tests which have passed (although the in process cluster seems to be extremely slow on my laptop so the number of tests included was limited). Further adjustments may be required to subset these tests for the automated github action checks if they take too long to execute.

Also includes minor updates to upstream proto dependencies and the over all flags for provenanced to fix missing log output.

Additional effort to follow separately for increased test coverage/refinement. This beta release is focused on getting initial features in the query api integrated for the Provenance Explorer team to work from.

@iramiller iramiller requested review from a user and channa-figure January 23, 2021 18:04
@iramiller iramiller requested a review from mtps as a code owner January 23, 2021 18:04
Copy link
Contributor

@channa-figure channa-figure left a comment

Choose a reason for hiding this comment

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

LGTM

@iramiller iramiller merged commit 27462b4 into main Jan 25, 2021
@iramiller iramiller deleted the add-marker-module branch January 25, 2021 17: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.

2 participants