-
Notifications
You must be signed in to change notification settings - Fork 3
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
GH actions config #29
Conversation
b406f52
to
5f5b955
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.
It makes sense to use self-hosted instances, we might get even more gains.
634817d
to
9ff0822
Compare
@jcortejoso Since switching to our runners, the e2e test fails because it cannot find the |
8bc8c40
to
9f1e92d
Compare
test test Update .github/workflows/ci.yaml Co-authored-by: Valentin Rodygin <[email protected]> Update .github/workflows/ci.yaml Co-authored-by: Valentin Rodygin <[email protected]>
9f1e92d
to
eb152fb
Compare
@jcortejoso @carterqw2 Interesting that our org runners are slower than the native GH runner. Should I switch back?
|
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.
Let's keep it, if we feel like it's a problem, I'm sure we can bump up our specs as needed.
eb152fb
to
257b1a0
Compare
@carterqw2 I removed the CircleCI config file now, so once we merge this we need to remove the requirements for circleCI runs as well (and I guess you're the only one with permissions to do so) |
Brings down runtime of tests from 12:30 to 9:30.