Skip to content

dev to main: ci(merge-bot): do not require python CodeQL analysis when no python files changed (+1 more) #24

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

Merged
patrick-main-promotion-bot merged 2 commits into from
Oct 11, 2026
Merged

dev to main: ci(merge-bot): do not require python CodeQL analysis when no python files changed (+1 more) #24

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
565e7fc
builder: pin Rust 1.96.1 to match upstream dbt-oss rust-toolchain
patrickryankenneth Oct 11, 2026
462ef7c
ci(merge-bot): do not require python CodeQL analysis when no python f…
patrickryankenneth Oct 11, 2026
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
6 changes: 6 additions & 0 deletions .github/workflows/merge-bot.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -141,12 +141,18 @@ jobs:
# aggregate check reports neutral while results are still being
# processed) mean "not settled yet": retry. Anything else that is
# not success is a hard fail. Timeout fails closed.
py_changed=$(gh api "repos/$GITHUB_REPOSITORY/pulls/${{ github.event.pull_request.number }}/files" --paginate --jq '.[].filename' | grep -cE '\.py$' || true)
echo "python files changed in PR: $py_changed"
for i in $(seq 1 20); do
ok=1
for name in "CodeQL" "Analyze (actions)" "Analyze (python)" "policy-ci"; do
c=$(gh api "repos/$GITHUB_REPOSITORY/commits/$HEAD_SHA/check-runs?per_page=100" \
--jq "[.check_runs[] | select(.name == \"$name\")] | sort_by(.started_at) | last | if . == null or .status != \"completed\" then \"none\" else .conclusion // \"none\" end")
echo "attempt $i: $name: $c"
if [ "$name" = "Analyze (python)" ] && [ "$py_changed" = "0" ] && [ "$c" = "none" ]; then
echo " no .py files changed, python analysis was not scheduled: not required"
continue
fi
case "$c" in
success) ;;
none|null|neutral) ok=0 ;;
Expand Down
2 changes: 1 addition & 1 deletion environments/alpine-3.24-musl/Containerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ ENV PROTOC=/usr/bin/protoc \
ARG RUSTUP_VERSION=1.29.1
ARG RUSTUP_SHA256_X86_64=331228566cc931f32cd684f9bebc5956a5e5d7394c8d5de595f196a658ab98e6
ARG RUSTUP_SHA256_AARCH64=1ddf36182ac5d1782dbeefcb9bea7f5f9412d88f4a0d5047b1564a223624e8c6
ARG RUST_VERSION=1.98.1
ARG RUST_VERSION=1.96.1

ENV RUSTUP_HOME=/opt/rustup \
CARGO_HOME=/opt/cargo \
Expand Down
Loading