Skip to content
This repository was archived by the owner on Jul 27, 2023. It is now read-only.

Include argument parentheses in range - #5

Merged
MichaReiser merged 1 commit into
mainfrom
include-parentheses-in-arguments-range
Jun 7, 2023
Merged

Include argument parentheses in range#5
MichaReiser merged 1 commit into
mainfrom
include-parentheses-in-arguments-range

Conversation

@MichaReiser

@MichaReiser MichaReiser commented Jun 6, 2023

Copy link
Copy Markdown
Member

This PR changes the parser to include the range of the opening and closing parentheses of function arguments in the arguments range.

@MichaReiser

MichaReiser commented Jun 6, 2023

Copy link
Copy Markdown
Member Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@MichaReiser
MichaReiser requested a review from charliermarsh June 6, 2023 09:43
@MichaReiser
MichaReiser marked this pull request as ready for review June 6, 2023 09:44
@MichaReiser
MichaReiser merged commit cfe0efc into main Jun 7, 2023
@MichaReiser
MichaReiser deleted the include-parentheses-in-arguments-range branch June 7, 2023 20:26
zanieb added a commit that referenced this pull request Jul 10, 2023
zanieb added a commit that referenced this pull request Jul 17, 2023
# Conflicts:
#	parser/src/parser.rs
#	parser/src/python.rs
#	parser/src/snapshots/rustpython_parser__parser__tests__decorator_ranges.snap
zanieb pushed a commit that referenced this pull request Jul 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant