Skip to content

Remove some unneccessary changes to reduce the diff from Stainless#673

Closed
adam-rl wants to merge 1 commit intomainfrom
adam/fix-smoketest-conflict
Closed

Remove some unneccessary changes to reduce the diff from Stainless#673
adam-rl wants to merge 1 commit intomainfrom
adam/fix-smoketest-conflict

Conversation

@adam-rl
Copy link
Copy Markdown
Contributor

@adam-rl adam-rl commented Nov 18, 2025

These diffs were reported by Stainless as custom code patches, which we are not interested in maintaining.

@adam-rl adam-rl requested a review from alb-rl November 18, 2025 20:27
run: ./scripts/bootstrap

- name: Run tests
run: ./scripts/test --ignore=tests/smoketests
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.

we sure this won't run the smoketests?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, we run with the default markers, which opt out of smoketest.

These diffs were reported by Stainless as custom code patches,
which we are not interested in maintaining.
@adam-rl adam-rl force-pushed the adam/fix-smoketest-conflict branch from ec272d6 to 84a9d4e Compare November 18, 2025 21:38
@adam-rl adam-rl closed this Feb 12, 2026
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