Skip to content

Add prop test to framing#2085

Open
Shourya742 wants to merge 2 commits intostratum-mining:mainfrom
Shourya742:2026-02-15-add-prop-test-to-framing
Open

Add prop test to framing#2085
Shourya742 wants to merge 2 commits intostratum-mining:mainfrom
Shourya742:2026-02-15-add-prop-test-to-framing

Conversation

@Shourya742
Copy link
Collaborator

closes: #2068

@Shourya742
Copy link
Collaborator Author

Blocked by: #2086

@Shourya742 Shourya742 force-pushed the 2026-02-15-add-prop-test-to-framing branch 2 times, most recently from 0502115 to a1523e4 Compare March 12, 2026 09:29
@Shourya742
Copy link
Collaborator Author

Blocked by: #2086

For now, I am not sure we should refactor the framing now or after the release. I have temporarily ignored the tests related to size_hint that were failing in order to unblock the PR. Once the relevant changes or refactors are introduced, we can revisit this and remove the ignore from those tests. Opening it for review.

@Shourya742 Shourya742 marked this pull request as ready for review March 12, 2026 13:52
@Shourya742 Shourya742 force-pushed the 2026-02-15-add-prop-test-to-framing branch from a1523e4 to 372d2a2 Compare March 23, 2026 11:29
@Shourya742 Shourya742 force-pushed the 2026-02-15-add-prop-test-to-framing branch from 372d2a2 to ba274db Compare March 23, 2026 11:30
Copy link
Collaborator

@lucasbalieiro lucasbalieiro left a comment

Choose a reason for hiding this comment

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

tACK ba274db

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.

Introduce property tests for framing-sv2

2 participants