search

envistaInteractive / eslint-config-enspire

ESLint configuration used at Enspire Commerce
http://www.enspirecommerce.com/
2 stars 0 forks source link

Rules fixes #11

Closed onel0p3z closed 7 years ago

onel0p3z commented 7 years ago

Brings the changes from:

onel0p3z commented 7 years ago

pinging @envistaInteractive/nodejs-admin to merge PR. 👍

crtpoms commented 7 years ago

Please remove my email from this.

On Aug 14, 2017, at 11:16 AM, Juan Lopez notifications@github.com wrote:

Brings the changes from:

dot-notation fixes #5 https://github.com/envistaInteractive/eslint-config-enspire/pull/5 You can view, comment on, or merge this pull request online at:

https://github.com/envistaInteractive/eslint-config-enspire/pull/11 https://github.com/envistaInteractive/eslint-config-enspire/pull/11 Commit Summary

fix: new config of dot-notation Default value config of dot-notation Test dot-notation and General structure test Remove comments Fix message of eslint to test Default settings eslint in test chore: apply feedback 1 Merge pull request #5 from envistaInteractive/fix/dot-notation File Changes

M configs/common.js https://github.com/envistaInteractive/eslint-config-enspire/pull/11/files#diff-0 (2) M package.json https://github.com/envistaInteractive/eslint-config-enspire/pull/11/files#diff-1 (2) A test/common.js https://github.com/envistaInteractive/eslint-config-enspire/pull/11/files#diff-2 (58) D tests/node.js https://github.com/envistaInteractive/eslint-config-enspire/pull/11/files#diff-3 (4) Patch Links:

https://github.com/envistaInteractive/eslint-config-enspire/pull/11.patch https://github.com/envistaInteractive/eslint-config-enspire/pull/11.patch https://github.com/envistaInteractive/eslint-config-enspire/pull/11.diff https://github.com/envistaInteractive/eslint-config-enspire/pull/11.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/envistaInteractive/eslint-config-enspire/pull/11, or mute the thread https://github.com/notifications/unsubscribe-auth/AHlAoAReE04u87m1YO7PN2fU1OZn9btFks5sYGTTgaJpZM4O2gfD.

onel0p3z commented 7 years ago

@crtpoms

You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#11>, or mute the thread https://github.com/notifications/unsubscribe-auth/AHlAoAReE04u87m1YO7PN2fU1OZn9btFks5sYGTTgaJpZM4O2gfD.