-
Notifications
You must be signed in to change notification settings - Fork 370
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.
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
chore: all feats, no feats, default feats top level workspace checks #1818
Conversation
SP1 Performance Test Results Branch: n/features-workflow
|
2144d18
to
87269dc
Compare
87269dc
to
f92a753
Compare
890271d
to
c714596
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I guess we could check the examples with this level of rigor too, but I think that's overkill.
.github/workflows/pr.yml
Outdated
@@ -186,7 +222,7 @@ jobs: | |||
|
|||
cli: | |||
name: CLI | |||
runs-on: [runs-on, runner=8cpu-linux-x64, "run-id=${{ github.run_id }}"] | |||
runs-on: [runs-on, runner=8cpu-linux-x64, disk=medium, "run-id=${{ github.run_id }}"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just curious what happened here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
checkout the current failing CI, its out of space on CLI job, maybe we need large here too?
@@ -44,7 +44,7 @@ enum-map = { version = "2.7.3", features = ["serde"] } | |||
test-artifacts = { workspace = true, optional = true } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i ran into this issue too lol
yea just don't merge until ci passes |
c5c777d
to
112d625
Compare
No description provided.