search

livepeer / protocol

Livepeer protocol
MIT License
154 stars 45 forks source link

Solidity version bump to 0.8.8 #537

Closed kautukkundan closed 2 years ago

kautukkundan commented 2 years ago

What does this pull request do? Explain your changes. (required)

upgrades solidity version to 0.8.8 for all the protocol contracts and test/mock contracts. Also edited a few tests in case the something changed. For SafeMath, I simply included an upgraded version of the library instead of using the native solidity math features. I felt that this was out of the scope of this PR and if needed could be done separately.

I have tried to combine related contracts together in a single commit for ease in review process and I suggest going over commits individually.

Most code changes are referenced from an earlier exercise at https://github.com/livepeer/protocol/pull/465

Specific updates (required)

How did you test each of these updates (required)

Does this pull request close any open issues?

related to https://github.com/code-423n4/2022-01-livepeer-findings/issues/10

Checklist:

kautukkundan commented 2 years ago

Will squash fixups before merge once 69ac6cf 51c2bff are good to go