Skip to content

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

Merged
patrick-main-promotion-bot[bot] merged 2 commits into
mainfrom
dev
Oct 11, 2026
Merged

patrick-main-promotion-bot[bot] merged 2 commits into
mainfrom
dev

Conversation

@patrick-dev-pr-bot

@patrick-dev-pr-bot patrick-dev-pr-bot Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

2 commit(s) ahead of main

builder: pin Rust 1.96.1 to match upstream dbt-oss rust-toolchain

(565e7fc)

ci(merge-bot): do not require python CodeQL analysis when no python files changed

(462ef7c)

@coderabbitai

coderabbitai Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 9fcd00ae-cf2d-4718-9b5f-2c2d1e43d00f

📥 Commits

Reviewing files that changed from the base of the PR and between 565e7fc and 462ef7c.


📒 Files selected for processing (1)
  • .github/workflows/merge-bot.yml

 ____________________________________________________________________
< Never go full rewrite. You don't buy that? Ask Netscape Navigator. >
 --------------------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 78698d74-3d1e-4601-aea7-6b5d4737b6b2


📥 Commits

Reviewing files that changed from the base of the PR and between 401152b and 565e7fc.



📒 Files selected for processing (1)
  • environments/alpine-3.24-musl/Containerfile


Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.




📝 Walkthrough
📝 Walkthrough

Walkthrough

The Alpine 3.24 musl Containerfile changes the pinned Rust toolchain version from 1.98.1 to 1.96.1. The rustup installation and version check use the configured version.

Changes

Rust toolchain pin

Layer / File(s) Summary
Update Rust version pin
environments/alpine-3.24-musl/Containerfile
The RUST_VERSION build argument changes from 1.98.1 to 1.96.1. rustup installs this version, and the rustc --version check verifies it.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Other

Suggested reviewers: patrickryankenneth



Merge Risk: ⚪ Minimal · up to 565e7

The Alpine builder now uses Rust 1.96.1, matching the stated upstream toolchain. Available dependency version requirements do not indicate a build incompatibility, so no material merge risk is identified.

Pre-merge checks | Passed 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: pinning Rust 1.96.1 to match the upstream dbt-oss rust-toolchain.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.



✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@deepsource-io

deepsource-io Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 401152b...462ef7c on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Python Oct 11, 2026 7:35p.m. Review ↗
Shell Oct 11, 2026 7:35p.m. Review ↗
Secrets Oct 11, 2026 7:35p.m. Review ↗
Docker Oct 11, 2026 7:35p.m. Review ↗
Rust Oct 11, 2026 7:35p.m. Review ↗
PowerShell Oct 11, 2026 7:35p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Deploying python-wheels-builder with  Cloudflare Pages  Cloudflare Pages

Latest commit: 462ef7c
Status: ✅  Deploy successful!
Preview URL: https://1276dc56.python-wheels-builder.pages.dev
Branch Preview URL: https://dev.python-wheels-builder.pages.dev

View logs

@patrickryankenneth
patrickryankenneth deployed to submit-pull-request October 11, 2026 19:35 — with GitHub Actions Active
@patrick-dev-pr-bot patrick-dev-pr-bot Bot changed the title builder: pin Rust 1.96.1 to match upstream dbt-oss rust-toolchain dev to main: ci(merge-bot): do not require python CodeQL analysis when no python files changed (+1 more) Oct 11, 2026
@patrick-main-promotion-bot
patrick-main-promotion-bot Bot merged commit 462ef7c into main Oct 11, 2026
17 of 18 checks passed

This branch was successfully deployed

3 active deployments
promote-to-main — 462ef7ce Deployed Oct 11, 2026 by patrickryankenneth via Fast-Forward to Main #67
submit-pull-request — 462ef7ce Deployed Oct 11, 2026 by patrickryankenneth via pr #6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant