Skip to content

Update src/rsz/test/repair_hold_multi_output_load.ok#10286

Merged
maliberty merged 1 commit intoThe-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:rsz-ok-update
Apr 28, 2026
Merged

Update src/rsz/test/repair_hold_multi_output_load.ok#10286
maliberty merged 1 commit intoThe-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:rsz-ok-update

Conversation

@openroad-ci
Copy link
Copy Markdown
Collaborator

Summary

Update the .ok

Type of Change

  • Bug fix

Impact

Fix CI

Verification

  • I have verified that the local build succeeds (./etc/Build.sh).
  • I have run the relevant tests and they pass.
  • My code follows the repository's formatting guidelines.
  • I have signed my commits (DCO).

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
@github-actions
Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the test expectation file repair_hold_multi_output_load.ok to reflect changes in timing results, including updates to WNS, slack values, and timing path details. I have no feedback to provide.

@maliberty maliberty merged commit 70bd4ff into The-OpenROAD-Project:master Apr 28, 2026
16 of 17 checks passed
@maliberty maliberty deleted the rsz-ok-update branch April 28, 2026 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants