Repository navigation
fix: preserve constructor context and streamline CI - #18
Merged
Merged
Conversation
cppbear
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Preserve constructor bodies when the declared return type matches the enclosing impl type, including compiler-recovered methods. Existing
Selfhandling remains; constructor discovery and alias resolution are unchanged.CI runs on PRs and main pushes, cancels superseded runs, and uses a lightweight patch check for documentation-only changes. The task regression keeps nine-function stress runs at N=1/default N=4 and five functions at N=2 (all exceeding 2N), and uses two functions for import handling and ordinary request failures. Queue, budget, serial compilation, and failure-cleanup checks remain.
Validation: the constructor regression fails before the fix and passes afterward; method-identity checks and a complete bytes analysis pass. Workflow parsing and 12 local scope cases pass. The shortened task suite and full Linux CI pass. No real model calls.