Skip to content

Commit b56a008

Browse files
committed
fix: properly handle bb path in ci after bbup installation
1 parent 23718d5 commit b56a008

File tree

2 files changed

+14
-16
lines changed

2 files changed

+14
-16
lines changed

.github/workflows/ci.yml

Lines changed: 12 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -110,22 +110,20 @@ jobs:
110110
uses: noir-lang/[email protected]
111111
with:
112112
toolchain: '1.0.0-beta.6'
113-
113+
114114
- name: Install Barretenberg
115+
shell: bash
115116
run: |
116-
# Check if bb is already available through Noir
117-
if ! command -v bb &> /dev/null; then
118-
echo "bb not found, installing..."
119-
curl -L https://raw.githubusercontent.com/AztecProtocol/aztec-packages/master/barretenberg/bbup/install | bash
120-
echo "$HOME/.bbup/bin" >> $GITHUB_PATH
121-
else
122-
echo "bb is already available at $(which bb)"
123-
fi
124-
125-
- name: Verify Barretenberg installation
126-
run: |
127-
bb --version || echo "bb version check failed, but continuing..."
128-
which bb || echo "bb not found in PATH"
117+
mkdir -p $HOME/.bb
118+
curl -L -o $HOME/.bb/bbup_installer https://raw.githubusercontent.com/AztecProtocol/aztec-packages/master/barretenberg/bbup/install
119+
echo "Installer downloaded..."
120+
chmod +x $HOME/.bb/bbup_installer
121+
echo "Installer made executable..."
122+
bash $HOME/.bb/bbup_installer
123+
echo "bbup installed..."
124+
echo "${HOME}/.bb" >> $GITHUB_PATH
125+
echo "Added ${HOME}/.bb to PATH"
126+
bash $HOME/.bb/bbup -nv 1.0.0-beta.6
129127
130128
- name: Install dependencies
131129
run: yarn install --frozen-lockfile

package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@
3535
"circuit:verify": "cd circuit && bb verify -k ./target/vk -p ./target/proof --oracle_hash keccak",
3636
"circuit:test": "cd circuit && nargo test",
3737
"contracts:deploy": "npx hardhat ignition deploy ./ignition/modules/SimpleCounter.ts --network sepolia --verify",
38-
"contracts:compile": "COMPILE=true npx hardhat compile",
39-
"contracts:test": "TEST=true npx hardhat test",
38+
"contracts:compile": "npx hardhat compile",
39+
"contracts:test": "npx hardhat test",
4040
"commitlint": "commitlint --edit",
4141
"commitlint:last": "commitlint --from HEAD~1 --to HEAD --verbose",
4242
"prepare": "husky"

0 commit comments

Comments
 (0)