Skip to content

test(utils): round-trip save_everything output against final_state - #514

Open
k-chrispens wants to merge 1 commit into
mainfrom
badtest/20261016-1525-modal
Open

k-chrispens wants to merge 1 commit into
mainfrom
badtest/20261016-1525-modal

Conversation

@k-chrispens

@k-chrispens k-chrispens commented Oct 24, 2026 •

Copy link
Copy Markdown
Collaborator

Improves these tests by adding round trip validation

@k-chrispens
k-chrispens requested a review from a team as a code owner October 24, 2026 00:00
Copilot AI balanced review requested due to automatic review settings October 24, 2026 00:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@coderabbitai

coderabbitai Bot commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 45 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 7e831697-c3ff-4ca8-9811-bfb157c7408e
📥 Commits

Reviewing files that changed from the base of the PR and between 0407ea4 and 1f79aff.

📒 Files selected for processing: 2
  • tests/integration/test_mismatch_integration.py
  • tests/utils/test_guidance_script_utils.py
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

This branch is waiting to be deployed

1 waiting deployment
gpu-testing — 1f79aff0 Waiting Oct 24, 2026 by k-chrispens via protenix-dev #971
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