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.

Sign up for GitHub

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

Jump to bottom

chore(deps): bump solidity version to 0.8.22 #246

Merged
merged 2 commits into from
Oct 30, 2023
Merged

chore(deps): bump solidity version to 0.8.22 #246

merged 2 commits into from
Oct 30, 2023

Conversation

rach-id
Copy link
Member

@rach-id rach-id commented Oct 26, 2023

Overview

Closes #245

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@rach-id rach-id requested a review from evan-forbes October 26, 2023 19:25
@rach-id rach-id self-assigned this Oct 26, 2023
@rach-id rach-id requested a review from adlerjohn as a code owner October 26, 2023 19:25
github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 26, 2023
View reviewed changes
src/DataRootTuple.sol
@@ -1,5 +1,5 @@
// SPDX-License-Identifier: Apache-2.0
pragma solidity ^0.8.21;
pragma solidity ^0.8.22;

Check warning

Code scanning / Slither

Incorrect versions of Solidity Warning

Pragma version^0.8.22 necessitates a version too recent to be trusted. Consider deploying with 0.8.18.
src/Constants.sol
@@ -1,5 +1,5 @@
// SPDX-License-Identifier: Apache-2.0
pragma solidity ^0.8.21;
pragma solidity ^0.8.22;

Check warning

Code scanning / Slither

Incorrect versions of Solidity Warning

Pragma version^0.8.22 necessitates a version too recent to be trusted. Consider deploying with 0.8.18.
src/Blobstream.sol Show resolved Hide resolved
evan-forbes
evan-forbes approved these changes Oct 30, 2023
View reviewed changes
src/Blobstream.sol Show resolved Hide resolved
@rach-id rach-id mentioned this pull request Oct 30, 2023
Open
@rach-id rach-id merged commit 9dade12 into celestiaorg:master Oct 30, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@evan-forbes evan-forbes evan-forbes approved these changes

@adlerjohn adlerjohn adlerjohn approved these changes

Assignees

@rach-id rach-id

Labels
chore dependencies
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Bump Solidity version to 0.8.22
3 participants
@rach-id @adlerjohn @evan-forbes