-
Notifications
You must be signed in to change notification settings - Fork 428
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
Abstract (start, end) from Spanning struct into new struct Span #1207
Conversation
1550f17
to
e775873
Compare
e775873
to
d814053
Compare
d814053
to
78f885b
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.
@audunhalland please, make this PR editable for maintainers:
The checkbox isn't visible for me. Could it be because the fork is owned by an organization in which I'm not an administrator? 🤔 Perhaps it's better if I re-create the PR from my personal fork. |
78f885b
to
70cd07b
Compare
New PR up at #1208 (from a my personal fork). Sorry for the noise 😿 The "Allow edits" github bug is discussed here: https://github.com/orgs/community/discussions/5634 |
@audunhalland thanks! |
This makes it easier to treat the source region (the Span) separately from the spanned item in
Spanning
.