Skip to content

Include end in nested module range.#14751

Merged
T-Gro merged 2 commits intodotnet:mainfrom
nojaf:nested-module-range
Feb 16, 2023
Merged

Include end in nested module range.#14751
T-Gro merged 2 commits intodotnet:mainfrom
nojaf:nested-module-range

Conversation

@nojaf
Copy link
Contributor

@nojaf nojaf commented Feb 15, 2023

Fixes #14749

@nojaf nojaf requested a review from a team as a code owner February 15, 2023 11:28
@T-Gro T-Gro merged commit 4247eb9 into dotnet:main Feb 16, 2023
Copy link
Member

@auduchinok auduchinok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nojaf Very nice change in the grammar, thank you!

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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Empty nested module range is in correct.

5 participants