Skip to content

Conversation

@XuehaiPan
Copy link
Collaborator

@XuehaiPan XuehaiPan commented Oct 9, 2025

As per the PR title, to alleviate loads for self-hosted runners.

Summary by CodeRabbit

  • Chores
    • Improved CI reliability by adding concurrency to auto-cancel in-progress runs when new PR updates arrive.
    • Enhanced workflow isolation with a dedicated virtual environment setup in automation tasks.
    • Expanded environment configuration with a new dependency index URL.
    • Standardized formatting and whitespace across workflows; no functional changes to jobs, tests, or permissions.
    • Documentation publishing and reminder workflows received non-functional formatting cleanups.

@github-actions
Copy link

github-actions bot commented Oct 9, 2025

👋 Hi! Thank you for contributing to the TileLang project.

Please remember to run bash format.sh in the root directory of the project to ensure your changes are properly linted and formatted. This will help ensure your contribution passes the format check.

We appreciate you taking this step! Our team will review your contribution, and we look forward to your awesome work!

🚀

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 9, 2025

Walkthrough

Adds concurrency controls to multiple CI workflows, introduces PYTORCH_INDEX_URL to AMD CI, and updates the bot workflow to build in a fresh virtual environment. Other files receive whitespace/formatting-only adjustments. No exported/public API changes.

Changes

Cohort / File(s) Summary of changes
CI concurrency controls
.github/workflows/ci.yml, .github/workflows/amd_ci.yml, .github/workflows/metal_ci.yml
Added top-level concurrency with group "${{ github.workflow }}-${{ github.ref }}" and cancel-in-progress for pull_request. In amd_ci, also added PYTORCH_INDEX_URL env. Minor whitespace/permissions formatting tweaks.
Bot workflow build setup
.github/workflows/bot.yml
In “Build original version”: added venv creation/activation, installed requirements and package after checkout. Minor trailing-space cleanup. Triggers remain under on:.
Formatting-only tweaks
.github/workflows/publish_docs.yml, .github/workflows/reminder.yml
Removed/moved blank lines and adjusted indentation; no logic changes.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as Developer
  participant GH as GitHub Actions
  participant CI as Workflow (ci/amd_ci/metal_ci)
  Note over GH,CI: Added concurrency controls (pull_request)
  Dev->>GH: Open/Update Pull Request
  GH->>CI: Trigger workflow
  alt Another run in same group is in progress
    GH-->>CI: Cancel in-progress run
    Note right of GH: cancel-in-progress: true (PR only)
  end
  CI->>CI: Execute jobs as defined
Loading
sequenceDiagram
  autonumber
  participant GH as GitHub Actions
  participant BOT as Workflow (bot.yml)
  participant Step as Build original version
  GH->>BOT: Trigger workflow
  BOT->>Step: Checkout main
  rect rgba(200,235,255,0.25)
    Note right of Step: New env setup
    Step->>Step: python -m venv tl
    Step->>Step: source tl/bin/activate
    Step->>Step: pip install -r requirements-test.txt
    Step->>Step: pip install .
  end
  BOT->>BOT: Continue perf/test and PR posting (unchanged)
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

A rabbit taps the CI drum, hop-hop, concise and neat,
Concurrency keeps the queues in time, no tangled feet.
A venv blooms where bots now build, clean as morning dew,
AMD hums with new index paths, streamlined through and through.
Whiskers twitch—merge night, bright blue! 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title accurately summarizes the primary change (automatic cancellation of in-progress CI runs on new commits), is concise, specific, and free of noise, matching the PR objectives.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f6d4bd3 and 926b9c8.

📒 Files selected for processing (6)
  • .github/workflows/amd_ci.yml (3 hunks)
  • .github/workflows/bot.yml (2 hunks)
  • .github/workflows/ci.yml (2 hunks)
  • .github/workflows/metal_ci.yml (2 hunks)
  • .github/workflows/publish_docs.yml (0 hunks)
  • .github/workflows/reminder.yml (1 hunks)
💤 Files with no reviewable changes (1)
  • .github/workflows/publish_docs.yml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: build-test-metal
  • GitHub Check: format-check
  • GitHub Check: format-check

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@LeiWang1999 LeiWang1999 merged commit 9a7cda4 into tile-ai:main Oct 9, 2025
7 of 8 checks passed
@XuehaiPan XuehaiPan deleted the ci-limit branch October 9, 2025 08:49
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.

2 participants