Skip to content

Fix a regression in the description of character classes in error message#197

Merged
hildjj merged 1 commit intopeggyjs:1.3from
Mingun:fix-message-regression
Oct 18, 2021
Merged

Fix a regression in the description of character classes in error message#197
hildjj merged 1 commit intopeggyjs:1.3from
Mingun:fix-message-regression

Conversation

@Mingun
Copy link
Copy Markdown
Member

@Mingun Mingun commented Oct 18, 2021

Regression was introduced in the fad4ab7

Character class description contains redundant commas between class parts:

[1,2,3-5] instead of [123-5] for class [123-5]

@Mingun Mingun force-pushed the fix-message-regression branch from 898dfe4 to 00790e1 Compare October 18, 2021 19:08
Character class description contains redundant commas between class parts:

[1,2,3-5] instead of [123-5] for class [123-5]
@Mingun Mingun force-pushed the fix-message-regression branch from 00790e1 to bd3798d Compare October 18, 2021 19:10
Copy link
Copy Markdown
Contributor

@hildjj hildjj left a comment

Choose a reason for hiding this comment

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

@Mingun Mingun changed the base branch from main to 1.3 October 18, 2021 20:20
@Mingun
Copy link
Copy Markdown
Member Author

Mingun commented Oct 18, 2021

Done

@hildjj hildjj merged commit 8bc052c into peggyjs:1.3 Oct 18, 2021
@Mingun Mingun deleted the fix-message-regression branch October 18, 2021 20:22
@Mingun Mingun mentioned this pull request May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants