Skip to content

Conversation

@nsaunders
Copy link
Owner

Description

This fixes the content of the error message that arises from duplicate properties or descriptors within a single ruleset.

For example, given this ruleset...

universal ? Rule.do
  width := nil
  width := nil -- Note the duplicate `width` property.

...the error currently reads as...

  Could not match type
           
    ( ... )
           
  with type
                          
    ( width :: CommonKeyword
    ...                   
    )  

After this change, it will read as...

  Could not match type
           
    ( ... )
           
  with type
                          
    ( width :: Measure Nil
    ...                   
    )  

...reflecting the real type of the property value.

Design considerations

N/A

Future plans

N/A

References

N/A

Code change checklist

  • Any new or updated functionality includes corresponding unit test coverage.
  • I have verified code formatting, run the unit tests, and checked for any changes in the examples.
  • I have added an entry to the Unreleased section of the CHANGELOG.

@nsaunders nsaunders merged commit 0204192 into master May 15, 2023
@nsaunders nsaunders added the defect Something isn't working label May 15, 2023
@nsaunders nsaunders deleted the feature/duplicate-error-fix branch July 27, 2023 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

defect Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants