Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Adding noParse on module #27

Merged
merged 1 commit into from
May 18, 2017
Merged

Conversation

paul-sachs
Copy link
Contributor

Related to #26

Will need to update @types once changed.

@paul-sachs paul-sachs force-pushed the noParse branch 2 times, most recently from ef829fe to 760670f Compare May 18, 2017 13:20
@paul-sachs
Copy link
Contributor Author

Little bit confused by errors in codacy, given the changes are consistent with existing code base.

Copy link
Member

@eliperelman eliperelman left a comment

Choose a reason for hiding this comment

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

Just 1 nit, otherwise looks good!

src/Module.js Outdated
@@ -33,6 +36,9 @@ module.exports = class extends ChainedMap {
.keys(value)
.forEach(name => this.rule(name).merge(value[name]));
}
case 'noParse': {
Copy link
Member

Choose a reason for hiding this comment

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

Insert 1 blank line before this case.

@eliperelman
Copy link
Member

@psachs21 don't worry about the codacy errors, I will get this merged. 😃

@eliperelman eliperelman merged commit 09d7f29 into neutrinojs:master May 18, 2017
@paul-sachs
Copy link
Contributor Author

@eliperelman thanks!

@eliperelman
Copy link
Member

Released in v3.3.0! Thank you!

@paul-sachs paul-sachs deleted the noParse branch May 24, 2017 14:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants