-
Notifications
You must be signed in to change notification settings - Fork 248
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
ci: Migrate to GH Actions for release #4947
Conversation
|
CodSpeed Performance ReportMerging #4947 will not alter performanceComparing Summary
|
WASM Query Engine file Size
|
/integration |
Do not merge until Buildkite pipelines are disabled. |
c608420
to
f52f86e
Compare
@@ -900,7 +900,7 @@ mod many_count_rel { | |||
// Nullable counts should be COALESCE'd to 0. | |||
insta::assert_snapshot!( | |||
run_query!(&runner, r#"{ | |||
findManyPost { | |||
findManyPost(orderBy: {id: "asc"}) { |
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.
Without orderBy
, this produced different order on Postgres 9
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.
Amazing work 🎉 cc @Jolg42
e582bc1
to
8ca8eea
Compare
[integration]
[integration]
[integration]
[integration]
[integration]
[integration]
[integration]
[integration]
[integration]
Moves and finishes the work Joël did in https://github.com/prisma/prisma-engines-builds.
For release process, this gets rid of both buildkite and engineer.
In order to migrate all tests that were running on buildkite to GH Actions we had to into nested job per connector - otherwise, matrix exceeded 256 jobs limit.
Otherwise, it works more or less identical to how engineer did with a couple of exceptions:
[integration]
into the commit message.