Skip to content

[Skip ci] Add checked-literals WIP#3176

Draft
rowanG077 wants to merge 1 commit intomasterfrom
checked-literals
Draft

[Skip ci] Add checked-literals WIP#3176
rowanG077 wants to merge 1 commit intomasterfrom
checked-literals

Conversation

@rowanG077
Copy link
Copy Markdown
Member

This introduces the brand new https://github.com/clash-lang/checked-literals plugin that makes sure literals in the code are proven to fit into the bounds of the type.

Still WIP because the package is not released but all instances are there.

Still TODO:

  • Write a changelog entry (see changelog/README.md)
  • Check copyright notices are up to date in edited files

@martijnbastiaan
Copy link
Copy Markdown
Member

What do you think of making the tests similar to checked-literals and part of clash-prelude:unittests? I can do that if you want.

@rowanG077
Copy link
Copy Markdown
Member Author

We can do that. But I would not remove the tests in checked-literals itself. This PR itself is still a WIP. It was quickly moving towards seeing if tests pass :)

@martijnbastiaan
Copy link
Copy Markdown
Member

Ok! I'll port it over. Yeah, I won't remove the tests in checked-literals itself.

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