Skip to content

[v2] Error on unresolvable or invalid inheritance bases#1830

Merged
vladimirradosavljevic merged 1 commit into
mainfrom
sdr23
Jun 8, 2026
Merged

[v2] Error on unresolvable or invalid inheritance bases#1830
vladimirradosavljevic merged 1 commit into
mainfrom
sdr23

Conversation

@vladimirradosavljevic

@vladimirradosavljevic vladimirradosavljevic commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Add a diagnostic when encountering unresolvable or invalid inheritance bases.
Adds two diagnostic groups: Resolution (IdentifierNotFound) for bases that don't resolve, and Type System (InvalidBase) for bases that resolve to a non-contract/interface (e.g. a library, struct, or free function).

@vladimirradosavljevic vladimirradosavljevic requested review from a team as code owners June 5, 2026 11:29
@changeset-bot

changeset-bot Bot commented Jun 5, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 15febda

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vladimirradosavljevic vladimirradosavljevic force-pushed the sdr23 branch 2 times, most recently from e105bc5 to 5adc46d Compare June 5, 2026 14:20
Comment thread crates/solidity-v2/outputs/cargo/semantic/src/passes/tests/binder.rs Outdated
Base automatically changed from sdr20 to main June 8, 2026 09:46
@vladimirradosavljevic vladimirradosavljevic force-pushed the sdr23 branch 2 times, most recently from 6a2da44 to 7e22200 Compare June 8, 2026 11:00
Add a diagnostic when encountering unresolvable or invalid inheritance
bases.
Adds two diagnostic groups: Resolution (IdentifierNotFound) for bases
that don't resolve, and Type System (InvalidBase) for bases that resolve
to a non-contract/interface (e.g. a library, struct, or free function).

Signed-off-by: Vladimir Radosavljevic <vr@matterlabs.dev>
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

🐰 Bencher Report

Branchsdr23
Testbedci

⚠️ WARNING: Truncated view!

The full continuous benchmarking report exceeds the maximum length allowed on this platform.

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

🐰 View full continuous benchmarking report in Bencher

Merged via the queue into main with commit 7ff125a Jun 8, 2026
17 checks passed
@vladimirradosavljevic vladimirradosavljevic deleted the sdr23 branch June 8, 2026 11:39
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