Skip to content

Commit 6837801

Browse files
authored
👷‍♂️ Update ci (#1468)
1 parent 834bbc4 commit 6837801

File tree

6 files changed

+36
-32
lines changed

6 files changed

+36
-32
lines changed

.github/workflows/ci-all-via-ir.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
profile: [via-ir-0,via-ir-1,via-ir-2,via-ir-3]
1414

1515
steps:
16-
- uses: actions/checkout@v4
16+
- uses: actions/checkout@v5
1717

1818
- name: Install Foundry
1919
uses: foundry-rs/foundry-toolchain@v1

.github/workflows/ci-invariant-intense.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
profile: [invariant-intense-0,invariant-intense-1,invariant-intense-2,invariant-intense-3]
1414

1515
steps:
16-
- uses: actions/checkout@v4
16+
- uses: actions/checkout@v5
1717

1818
- name: Install Foundry
1919
uses: foundry-rs/foundry-toolchain@v1

.github/workflows/ci-super-intense.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
profile: [super-intense-0,super-intense-1]
1414

1515
steps:
16-
- uses: actions/checkout@v4
16+
- uses: actions/checkout@v5
1717

1818
- name: Install Foundry
1919
uses: foundry-rs/foundry-toolchain@v1

.github/workflows/ci-wake.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
profile: [via-ir-off, via-ir-on]
1414

1515
steps:
16-
- uses: actions/checkout@v4
16+
- uses: actions/checkout@v5
1717

1818
- name: Setup Wake
1919
uses: Ackee-Blockchain/[email protected]

.github/workflows/ci.yml

Lines changed: 29 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ jobs:
1818
matrix:
1919
profile: [post-cancun,post-cancun-via-ir,solc-past-versions-0,solc-past-versions-1,via-ir,min-solc,min-solc-via-ir,intense]
2020
steps:
21-
- uses: actions/checkout@v4
21+
- uses: actions/checkout@v5
2222
- name: Install Foundry Stable
2323
uses: foundry-rs/foundry-toolchain@v1
2424
with:
@@ -78,27 +78,30 @@ jobs:
7878
forge test --fuzz-runs 5000
7979
)
8080
81-
tests-ithaca:
82-
name: Forge Ithaca Testing
83-
runs-on: ubuntu-latest
84-
strategy:
85-
matrix:
86-
profile: [post-cancun,post-cancun-via-ir]
87-
steps:
88-
- uses: actions/checkout@v4
89-
- name: Install Foundry Nightly
90-
uses: foundry-rs/foundry-toolchain@v1
91-
with:
92-
version: stable
93-
- name: Install Dependencies
94-
run: forge install
95-
- name: Run Tests with ${{ matrix.profile }}
96-
run: |
97-
if [[ "${{ matrix.profile }}" == "post-cancun" ]]; then
98-
FOUNDRY_PROFILE=ithaca forge test --use 0.8.28
99-
elif [[ "${{ matrix.profile }}" == "post-cancun-via-ir" ]]; then
100-
FOUNDRY_PROFILE=ithaca forge test --use 0.8.28 --via-ir
101-
fi
81+
# Disable for now until we can use `foundry.toml` setting to override the
82+
# transaction gas limit. See: EIP-7825.
83+
#
84+
# tests-ithaca:
85+
# name: Forge Ithaca Testing
86+
# runs-on: ubuntu-latest
87+
# strategy:
88+
# matrix:
89+
# profile: [post-cancun,post-cancun-via-ir]
90+
# steps:
91+
# - uses: actions/checkout@v5
92+
# - name: Install Foundry Nightly
93+
# uses: foundry-rs/foundry-toolchain@v1
94+
# with:
95+
# version: nightly
96+
# - name: Install Dependencies
97+
# run: forge install
98+
# - name: Run Tests with ${{ matrix.profile }}
99+
# run: |
100+
# if [[ "${{ matrix.profile }}" == "post-cancun" ]]; then
101+
# FOUNDRY_PROFILE=ithaca forge test --use 0.8.28
102+
# elif [[ "${{ matrix.profile }}" == "post-cancun-via-ir" ]]; then
103+
# FOUNDRY_PROFILE=ithaca forge test --use 0.8.28 --via-ir
104+
# fi
102105

103106
codespell:
104107
runs-on: ${{ matrix.os }}
@@ -108,7 +111,7 @@ jobs:
108111
- ubuntu-latest
109112
steps:
110113
- name: Checkout
111-
uses: actions/checkout@v4
114+
uses: actions/checkout@v5
112115
- name: Run codespell
113116
uses: codespell-project/[email protected]
114117
with:
@@ -126,7 +129,7 @@ jobs:
126129
- ubuntu-latest
127130
steps:
128131
- name: Checkout code
129-
uses: actions/checkout@v4
132+
uses: actions/checkout@v5
130133
- name: Install Foundry
131134
uses: foundry-rs/foundry-toolchain@v1
132135
with:
@@ -150,11 +153,11 @@ jobs:
150153
- ubuntu-latest
151154
steps:
152155
- name: Checkout code
153-
uses: actions/checkout@v4
156+
uses: actions/checkout@v5
154157
- name: Install Foundry
155158
uses: foundry-rs/foundry-toolchain@v1
156159
with:
157-
version: stable
160+
version: nightly
158161
- name: Install Dependencies
159162
run: forge install
160163
- name: Run prep scripts and forge fmt

src/utils/MerkleTreeLib.sol

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -262,8 +262,9 @@ library MerkleTreeLib {
262262
mstore(0x00, 0xe7171dc4) // `MerkleTreeLeavesEmpty()`.
263263
revert(0x1c, 0x04)
264264
}
265-
let p := 1 // Padded length.
266-
for {} lt(p, l) {} { p := add(p, p) }
265+
let p := sub(mload(leaves), 1)
266+
for { let i := 1 } lt(i, 0x80) { i := add(i, i) } { p := or(p, shr(i, p)) }
267+
p := add(p, 1)
267268
mstore(result, p) // Store length.
268269
mstore(0x40, add(result, add(0x20, shl(5, p)))) // Allocate memory.
269270
let d := sub(result, leaves)

0 commit comments

Comments
 (0)