-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add type coverage #889
Add type coverage #889
Conversation
Damn, I clicked on update branch and merged main into branch... sorry, let's check if there is way to disable this button, or change it's behavior. I will fix this error in functional tests. |
a984e9f
to
c7f0205
Compare
I've fixed this now: git switch Feature/Add-type-coverage
git reset --hard a14f676
git rebase
git push -f |
I changed it to 100, 100 and 95 for properties. We can update it to 100, but first we need to add property type. This should work for now. Let's do it in another PR. |
04661a3
to
3d85f61
Compare
3d85f61
to
07e9b20
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. It should get an entry in the changelog, though.
1956c84
to
937f955
Compare
Resolves: #830