-
Notifications
You must be signed in to change notification settings - Fork 185
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
check the whole caret position in the testkit #1259
check the whole caret position in the testkit #1259
Conversation
ca99e78
to
5ca9257
Compare
5ca9257
to
367c004
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.
Replace the example in the documentation by a real one from the tests
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.
LGTM 👍 Would be good to mention this in the v1.0 release notes as a breaking change.
@olafurpg I have added the 1.0.0 milestone on this PR and added a bullet point in the outstanding tasks for the 1.0.0 release. Can you think of any other breaking change we should document so we can tag it as well? |
This pull request ensures that the testkit checks the whole caret (
^^^^
) position range. Before only the range start was tested.