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

Update Parity version for integration testing to 2.3.5 #1292

Merged
merged 1 commit into from
Mar 21, 2019

Conversation

njgheorghita
Copy link
Contributor

@njgheorghita njgheorghita commented Mar 20, 2019

What was wrong?

Updated Parity version to 2.3.5 for integration tests

  • deleted .travis.yml
  • deleted parity fixture data for 1.8.7 and 1.8.8

Cute Animal Picture

image

@njgheorghita njgheorghita force-pushed the upgrade-parity branch 8 times, most recently from 96830b6 to 75bf7c8 Compare March 20, 2019 16:19
@njgheorghita njgheorghita changed the title dirty Update Parity version for integration testing to 2.3.5 Mar 20, 2019
Copy link
Collaborator

@kclowes kclowes left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for picking this up!

'--rpcport', parity_port,
'--jsonrpc-apis', 'all',
'--jsonrpc-port', parity_port,
'--fat-db', 'on',
# '--author', common.COINBASE[2:],
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we can probably get rid of this

'--rpcport', parity_port,
'--jsonrpc-apis', 'all',
'--jsonrpc-port', parity_port,
'--fat-db', 'on',
# '--author', common.COINBASE[2:],
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same here, let's get rid of this

Copy link
Collaborator

@kclowes kclowes left a comment

Choose a reason for hiding this comment

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

:shipit: Thanks for taking this on!

@kclowes
Copy link
Collaborator

kclowes commented Mar 20, 2019

Ha, forgot I had already submitted my review. Double approval!

@njgheorghita njgheorghita merged commit ef728a8 into ethereum:master Mar 21, 2019
@njgheorghita njgheorghita deleted the upgrade-parity branch March 21, 2019 12:18
@kclowes kclowes mentioned this pull request Aug 7, 2019
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