Skip to content

Conversation

esdrubal
Copy link
Contributor

@esdrubal esdrubal commented Sep 9, 2024

Description

This PR adds a test that proves implicit trait constraint already returns
an error. The method dummy ICE reported in #6377 was fixed by #6490.

Closes #6377.

Checklist

  • I have linked to any relevant issues.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have updated the documentation where relevant (API docs, the reference, and the Sway book).
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
  • I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
  • I have requested a review from the relevant team or maintainers.

This PR adds test that proves implicit trait constraint already returns
 an error. The method dummy ICE reported in #6377 was fixed by #6490.

Closes #6377.
@esdrubal esdrubal added bug Something isn't working P: medium audit-report Related to the audit report labels Sep 9, 2024
@esdrubal esdrubal self-assigned this Sep 9, 2024
@esdrubal esdrubal requested a review from a team as a code owner September 9, 2024 15:38
@esdrubal esdrubal enabled auto-merge (squash) September 16, 2024 12:01
@esdrubal esdrubal merged commit ea4b9e9 into master Sep 16, 2024
34 checks passed
@esdrubal esdrubal deleted the esdrubal/test_implicit_trait_constraint branch September 16, 2024 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

audit-report Related to the audit report bug Something isn't working P: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implicit Trait Constraints

4 participants