microsoft / microsoft/powerquery-parser
[BUG] Invalid identifier allowed as record field name
Open
@JordanBoltonMN is already working on this.
Since Jul 20, 2021.
bug
- Dominant language
- TypeScript
- Stars
- 137
- Forks
- 34
- Avg merge
- 50m
- Merged PRs (30d)
- 1
Description
Expected behavior
To align with Query Editor's behavior, in [1.=true], "1." should be rejected as an invalid identifier.
Actual behavior
The parser (as tested in VSCode using the Power Query / M Language extension) accepts the above as valid M.
Additional context
https://github.com/MicrosoftDocs/powerquery-docs/issues/30 discusses the grammar for a generalized identifier.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.