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

+ Implement capturePos rule #152

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@alexander-myltsev
Contributor

alexander-myltsev commented Jul 26, 2015

No description provided.

@alexander-myltsev

This comment has been minimized.

Show comment
Hide comment
@alexander-myltsev

alexander-myltsev Jul 26, 2015

Contributor

@sirthias Mathias, would you merge that facility? I would add tests if yes.

Contributor

alexander-myltsev commented Jul 26, 2015

@sirthias Mathias, would you merge that facility? I would add tests if yes.

@sirthias

This comment has been minimized.

Show comment
Hide comment
@sirthias

sirthias Jul 27, 2015

Owner

Is this really something we should provide as a predefined something?
What about line numbers and column numbers?
I could imagine that different applications have different needs with regard to position tracking.

Owner

sirthias commented Jul 27, 2015

Is this really something we should provide as a predefined something?
What about line numbers and column numbers?
I could imagine that different applications have different needs with regard to position tracking.

@alexander-myltsev

This comment has been minimized.

Show comment
Hide comment
@alexander-myltsev

alexander-myltsev Jul 27, 2015

Contributor

OK, I am convinced. Closing PR.

Contributor

alexander-myltsev commented Jul 27, 2015

OK, I am convinced. Closing PR.

@alexander-myltsev alexander-myltsev deleted the alexander-myltsev:capturepos-rule branch Jul 27, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment