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

Remove pre-OP_RETURN legacy code #343

Merged
merged 1 commit into from Feb 15, 2016

Conversation

Projects
None yet
2 participants
@dexX7
Copy link
Member

dexX7 commented Feb 12, 2016

Because the network upgraded to Omni Core 0.0.10, the old legacy code is no longer needed.

I tested it by adding a new checkpoint for block 398075 and parsing fresh with -startclean.

Remove pre-OP_RETURN legacy code
Because the network upgraded to Omni Core 0.0.10, the old legacy code is no longer needed.

@dexX7 dexX7 added the consensus label Feb 12, 2016

@dexX7 dexX7 added this to the 0.0.10.1 milestone Feb 12, 2016

@zathras-crypto

This comment has been minimized.

Copy link

zathras-crypto commented Feb 14, 2016

Reviewed, looks good to me... Tested, confirmed matching consensus hash with 0.0.10 also (as of block 398363).

OK to merge :)

@dexX7 dexX7 added the status: ready label Feb 15, 2016

@dexX7 dexX7 merged commit 77b2996 into OmniLayer:omnicore-0.0.10 Feb 15, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

dexX7 added a commit that referenced this pull request Feb 15, 2016

Merge pull request #343
77b2996 Remove pre-OP_RETURN legacy code (dexX7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment