Skip to content

Commit

Permalink
Merge pull request #51 from hasura/djh/add-merge-groups
Browse files Browse the repository at this point in the history
Enable merge groups for merge queue
  • Loading branch information
danieljharvey authored Oct 31, 2023
2 parents 2050264 + ee504f7 commit 8da0175
Show file tree
Hide file tree
Showing 8 changed files with 26 additions and 19 deletions.
8 changes: 5 additions & 3 deletions .github/workflows/cargo-build.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
on: [push]
on:
push:
merge_group:

name: SQLServer NDC build

Expand Down Expand Up @@ -38,7 +40,7 @@ jobs:
with:
status: ${{ job.status }}
notify_when: failure
notification_title: '😧 Error on <{repo_url}|{repo}>'
message_format: '🐴 *{workflow}* {status_message} for <{repo_url}|{repo}>'
notification_title: "😧 Error on <{repo_url}|{repo}>"
message_format: "🐴 *{workflow}* {status_message} for <{repo_url}|{repo}>"
env:
SLACK_WEBHOOK_URL: ${{ secrets.BROKEN_BUILD_SLACK_WEBHOOK_URL }}
8 changes: 5 additions & 3 deletions .github/workflows/cargo-test.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
on: [push]
on:
push:
merge_group:

name: SQLServer NDC tests

Expand Down Expand Up @@ -48,7 +50,7 @@ jobs:
with:
status: ${{ job.status }}
notify_when: failure
notification_title: '😧 Error on <{repo_url}|{repo}>'
message_format: '🐴 *{workflow}* {status_message} for <{repo_url}|{repo}>'
notification_title: "😧 Error on <{repo_url}|{repo}>"
message_format: "🐴 *{workflow}* {status_message} for <{repo_url}|{repo}>"
env:
SLACK_WEBHOOK_URL: ${{ secrets.BROKEN_BUILD_SLACK_WEBHOOK_URL }}
1 change: 1 addition & 0 deletions .github/workflows/nix-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ name: nix check

on:
push:
merge_group:

jobs:
nix-develop:
Expand Down
6 changes: 1 addition & 5 deletions .github/workflows/nix-deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,7 @@ name: multi-architecture docker build

on:
push:
branches:
- main
- djh/multi-arch-build
tags:
- "v*"
merge_group:

jobs:
build_and_deploy:
Expand Down
7 changes: 3 additions & 4 deletions .github/workflows/rustfmt.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ name: SQLServer NDC check formatting with Rustfmt

on:
push:
merge_group:

jobs:
formatting:
Expand All @@ -27,9 +28,7 @@ jobs:
with:
status: ${{ job.status }}
notify_when: failure
notification_title: '😧 Error on <{repo_url}|{repo}>'
message_format: '🐴 *{workflow}* {status_message} for <{repo_url}|{repo}>'
notification_title: "😧 Error on <{repo_url}|{repo}>"
message_format: "🐴 *{workflow}* {status_message} for <{repo_url}|{repo}>"
env:
SLACK_WEBHOOK_URL: ${{ secrets.BROKEN_BUILD_SLACK_WEBHOOK_URL }}


4 changes: 2 additions & 2 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 9 additions & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,12 @@ members = [
"crates/ndc-sqlserver",
"crates/query-engine",
]


# Temporary patch to get the build working. We shall remove this once a new
# version of ndc-spec is published, and ndc-sdk is updated accordingly.
# The "//" is to convince Cargo to accept this, due to
# https://github.com/rust-lang/cargo/issues/10756 .
[patch."https://github.com/hasura/ndc-spec.git"]
ndc-client = { git = "https://github.com/hasura//ndc-spec.git", rev = "3778605" }
ndc-test = { git = "https://github.com/hasura//ndc-spec.git", rev = "3778605" }
2 changes: 0 additions & 2 deletions flake.nix
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,6 @@
crane = {
url = "github:ipetkov/crane";
inputs.nixpkgs.follows = "nixpkgs";
inputs.rust-overlay.follows = "rust-overlay";
inputs.flake-utils.follows = "flake-utils";
};

rust-overlay = {
Expand Down

0 comments on commit 8da0175

Please sign in to comment.