Skip to content
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

Fix most of the typos in the comments #1838

Closed
wants to merge 2 commits into from
Closed

Fix most of the typos in the comments #1838

wants to merge 2 commits into from

Conversation

ly0va
Copy link

@ly0va ly0va commented Aug 5, 2021

No description provided.

@ricmoo ricmoo added documentation Documentation related issue. on-deck This Enhancement or Bug is currently being worked on. labels Aug 9, 2021
@@ -1277,8 +1277,8 @@ type _Location = {
export function parse(code: string, options?: ParserOptions): Node {
if (options == null) { options = { }; }

// Since jison allows \n, \r or \r\n line endings, we need some
Copy link
Member

Choose a reason for hiding this comment

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

Jison is the application used to process the file (a JavaScript equivalent to Bison). ;)

Copy link
Author

Choose a reason for hiding this comment

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

my bad 🙃

Copy link
Member

Choose a reason for hiding this comment

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

No worries. I just finished going through your PR and merging it locally.

Thanks a lot for your attention to detail. :)

ricmoo added a commit that referenced this pull request Oct 16, 2021
@ricmoo
Copy link
Member

ricmoo commented Oct 20, 2021

Merged.

Thanks! :)

@ricmoo ricmoo closed this Oct 20, 2021
@ricmoo ricmoo added fixed/complete This Bug is fixed or Enhancement is complete and published. and removed on-deck This Enhancement or Bug is currently being worked on. labels Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation related issue. fixed/complete This Bug is fixed or Enhancement is complete and published.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants