Skip to content

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Dec 24, 2025

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

@augmentcode
Copy link

augmentcode bot commented Dec 24, 2025

🤖 Augment PR Summary

Summary: Updates the compiler pipeline to explicitly capture the outcome of schema transformation passes.
Changes: Adds <cassert> and uses [[maybe_unused]] variables plus assertions around SchemaTransformer::apply() calls to avoid unused/ignored-result warnings.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti force-pushed the maybe-unused-warnings branch from a1ed27c to e2ef916 Compare December 24, 2025 18:13
@jviotti jviotti merged commit 8fae212 into main Dec 24, 2025
13 checks passed
@jviotti jviotti deleted the maybe-unused-warnings branch December 24, 2025 18:15
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