Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[SC-BOT] Update opensta #4400

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[SC-BOT] Update opensta #4400

wants to merge 1 commit into from

Conversation

@siliconcompiler-bot
Copy link

@siliconcompiler-bot siliconcompiler-bot commented Nov 6, 2025 •
edited by coderabbitai bot
Loading

Updating opensta from 423c12c8f9d66ad0093f58a5c997b241394b334b to 79d8a5ecb32e9b6b02fe311ed29e6a652c407cf6
Check parallaxsw/OpenSTA@423c12c...79d8a5e for changes

Summary by CodeRabbit

  • Chores
    • Updated OpenSTA tool reference to latest version.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 6, 2025 •
edited
Loading

Walkthrough

This PR updates the pinned OpenSTA tool's git-commit reference in the tools configuration file from one commit hash to another, reflecting a version bump for the OpenSTA dependency.

Changes

Cohort / File(s) Change summary
OpenSTA dependency version pin
siliconcompiler/toolscripts/_tools.json		 Updated OpenSTA git-commit from 423c12c8f9d66ad0093f58a5c997b241394b334b to 79d8a5ecb32e9b6b02fe311ed29e6a652c407cf6

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

This is a single-line configuration update with no logic changes, structural modifications, or side effects.

Possibly related PRs

  • PR #4308: Updates the same OpenSTA git-commit entry in siliconcompiler/toolscripts/_tools.json
  • PR #4393: Previously pinned the OpenSTA commit to the hash this PR is moving from (423c12c...)
  • PR #4210: Also updates the OpenSTA git-commit field in the tools configuration to a different target revision

Poem

🐰 Hop, hop, the version hops ahead,
A new commit hash, the old one's shed,
OpenSTA's wheels now spin so fast,
Configuration's fresh—the die is cast! ✨

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title '[SC-BOT] Update opensta' accurately reflects the main change: updating the OpenSTA tool version via a git-commit hash update in the tools.json file.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bot/opensta-update
📜 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 1c2cdde and 3a8ec07.

📒 Files selected for processing (1)
  • siliconcompiler/toolscripts/_tools.json (1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings 
Learnt from: gadfort
Repo: siliconcompiler/siliconcompiler PR: 4276
File: siliconcompiler/toolscripts/_tools.json:20-20
Timestamp: 2025-10-08T00:39:15.602Z
Learning: Bot-generated PRs from siliconcompiler-bot that update tool commits in `siliconcompiler/toolscripts/_tools.json` have pre-validated commits via automated scripts. Do not flag these updates as invalid unless there is clear evidence of an issue beyond verification method limitations.
📚 Learning: 2025-10-08T00:39:15.602Z 
Learnt from: gadfort
Repo: siliconcompiler/siliconcompiler PR: 4276
File: siliconcompiler/toolscripts/_tools.json:20-20
Timestamp: 2025-10-08T00:39:15.602Z
Learning: Bot-generated PRs from siliconcompiler-bot that update tool commits in `siliconcompiler/toolscripts/_tools.json` have pre-validated commits via automated scripts. Do not flag these updates as invalid unless there is clear evidence of an issue beyond verification method limitations.

Applied to files:

  • siliconcompiler/toolscripts/_tools.json
⏰ 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). (22)
  • GitHub Check: Generate cache
  • GitHub Check: Pure Python tests (3.13, ubuntu-latest)
  • GitHub Check: Pure Python tests (3.13, ubuntu-24.04-arm)
  • GitHub Check: Pure Python tests (3.13, macos-latest)
  • GitHub Check: Pure Python tests (3.9, macos-13)
  • GitHub Check: Pure Python tests (3.13, windows-latest)
  • GitHub Check: Pure Python tests (3.11, macos-latest)
  • GitHub Check: Pure Python tests (3.11, windows-latest)
  • GitHub Check: Pure Python tests (3.12, macos-latest)
  • GitHub Check: Pure Python tests (3.12, ubuntu-latest)
  • GitHub Check: Pure Python tests (3.12, ubuntu-24.04-arm)
  • GitHub Check: Pure Python tests (3.12, windows-latest)
  • GitHub Check: Pure Python tests (3.11, ubuntu-24.04-arm)
  • GitHub Check: Pure Python tests (3.11, ubuntu-latest)
  • GitHub Check: Pure Python tests (3.9, ubuntu-latest)
  • GitHub Check: Pure Python tests (3.10, macos-latest)
  • GitHub Check: Pure Python tests (3.9, windows-latest)
  • GitHub Check: Pure Python tests (3.10, ubuntu-24.04-arm)
  • GitHub Check: Pure Python tests (3.10, windows-latest)
  • GitHub Check: Pure Python tests (3.9, ubuntu-24.04-arm)
  • GitHub Check: Pure Python tests (3.10, ubuntu-latest)
  • GitHub Check: Pure Python tests with docs
🔇 Additional comments (1)
siliconcompiler/toolscripts/_tools.json (1)

18-22: OpenSTA commit update looks good.

The git-commit hash has been cleanly updated from a previous version to 79d8a5ecb32e9b6b02fe311ed29e6a652c407cf6. Both hashes are valid SHA-1 identifiers, the JSON structure is intact, and the configuration remains consistent. Based on learnings, bot-generated updates to tool commits in this file are pre-validated via automated scripts.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@siliconcompiler-bot