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

docs: add error message for cargo test on darwin #215

Merged
merged 8 commits into from
Jan 15, 2025
Merged

docs: add error message for cargo test on darwin #215

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
1d9fc5a
docs: add error message for `cargo test` on darwin
MegaRedHand Jan 14, 2025
24b4637
docs: fix command for pulling foundry image
MegaRedHand Jan 14, 2025
1746af8
ci: use ubuntu-22.04 to avoid docker error
MegaRedHand Jan 14, 2025
6f1f233
chore: sync two readmes
MegaRedHand Jan 14, 2025
f14f425
Revert "ci: use ubuntu-22.04 to avoid docker error"
MegaRedHand Jan 15, 2025
13e27c6
Merge branch 'main' into update-docker-pull-mention
MegaRedHand Jan 15, 2025
2dc5e58
docs: merge `cargo test` and `exclude` flag version
MegaRedHand Jan 15, 2025
b8db7ea
chore: sync two readmes
MegaRedHand Jan 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 5 additions & 6 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,20 +73,19 @@ To test locally :-
You need `foundry` to successfully run it.

```bash
cargo test
cargo test --workspace --exclude eigen-utils
```

On Mac, you might need to pull the foundry image manually specifying the platform with:
On Mac, you might get an error similar to:

```bash
docker pull --platform amd64 foundry-rs/foundry
Error starting anvil container: Client(PullImage { descriptor: "ghcr.io/foundry-rs/foundry:latest", err: DockerStreamError { error: "no matching manifest for linux/arm64/v8 in the manifest list entries" } })
```

Sometimes the bindings fail due to containing botched doctests.
You can exclude them from the test run with:
In that case, you'll need to pull the foundry image for the `amd64` platform with:

```bash
cargo test --workspace --exclude eigen-utils
docker pull --platform linux/amd64/v8 ghcr.io/foundry-rs/foundry:latest
```

At least 1 `approving` review is required to merge the PR.
Expand Down
11 changes: 5 additions & 6 deletions crates/eigensdk/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,20 +73,19 @@ To test locally :-
You need `foundry` to successfully run it.

```bash
cargo test
cargo test --workspace --exclude eigen-utils
```

On Mac, you might need to pull the foundry image manually specifying the platform with:
On Mac, you might get an error similar to:

```bash
docker pull --platform amd64 foundry-rs/foundry
Error starting anvil container: Client(PullImage { descriptor: "ghcr.io/foundry-rs/foundry:latest", err: DockerStreamError { error: "no matching manifest for linux/arm64/v8 in the manifest list entries" } })
```

Sometimes the bindings fail due to containing botched doctests.
You can exclude them from the test run with:
In that case, you'll need to pull the foundry image for the `amd64` platform with:

```bash
cargo test --workspace --exclude eigen-utils
docker pull --platform linux/amd64/v8 ghcr.io/foundry-rs/foundry:latest
```

At least 1 `approving` review is required to merge the PR.
Expand Down
Loading