Skip to content

ISSUE-345: fix invalid regular expression#346

Merged
mrmlnc merged 2 commits intomasterfrom
ISSUE-345_fix_invalid_regular_expression
Jan 7, 2022
Merged

ISSUE-345: fix invalid regular expression#346
mrmlnc merged 2 commits intomasterfrom
ISSUE-345_fix_invalid_regular_expression

Conversation

@mrmlnc
Copy link
Owner

@mrmlnc mrmlnc commented Jan 7, 2022

What is the purpose of this pull request?

This is a fix addressed to #345.

What changes did you make? (Give an overview)

  1. Require node.js 8.6.0. The micromatch package works only with node.js 8.6+. This is not a major change, since this version was previously required implicitly.
  2. Fix regression from the issue.

mrmlnc added 2 commits January 7, 2022 23:29
The `micromatch` package works only with Node.js 8.6+.
@mrmlnc mrmlnc merged commit 5ba78d1 into master Jan 7, 2022
@mrmlnc mrmlnc deleted the ISSUE-345_fix_invalid_regular_expression branch January 7, 2022 20:43
@alfaproject
Copy link

Not sure I understand the engine change? The bug report says it was running on node 16 o:

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