Skip to content
This repository has been archived by the owner. It is now read-only.

Improve compatiblity #56

Closed
wants to merge 3 commits into from
Closed

Improve compatiblity #56

wants to merge 3 commits into from

Conversation

@quanglam2807
Copy link

quanglam2807 commented Mar 24, 2017

Add .node to main file path: Small but important. Without this, ESLint will not detect ad-block correctly (https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-unresolved.md). Also, Spectron won't launch on macOS.

Remove preinstall: The packages are included in dependencies already. This script conflicts with electron-builder. https://ci.appveyor.com/project/webcatalog/webcatalog/build/55. Package stops working with this removal >

Update: Add `--ignore-scripts" solve the problems.

Small but important. Without this, ESLint will not detect ad-block correctly (https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-unresolved.md). Also, Spectron won't launch on macOS.
@quanglam2807 quanglam2807 changed the title Add .node to main file path Add .node to main file path & Remove preinstall Mar 24, 2017
@quanglam2807 quanglam2807 changed the title Add .node to main file path & Remove preinstall Add .node to main file path & Add --ignore-scripts to preinstall Mar 24, 2017
@quanglam2807 quanglam2807 changed the title Add .node to main file path & Add --ignore-scripts to preinstall Improve compatiblity Mar 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.