Skip to content

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!

'--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

'--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