Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Adds test for implicit trait constraint. #6517

Merged
merged 5 commits into from
Sep 16, 2024

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
# for free to join this conversation on GitHub. Already have an account? # 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