Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

String: Updated mochify version in package.json #222

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ossdev07
Copy link

@ossdev07 ossdev07 commented Oct 9, 2018

PhantomJS is an unmaintained project, removed the optional dependency of phantomJS from mochify to use headlesschrome.
Updated node version in travis.yml because earlier version used were very old.

Signed-off-by: ossdev07 [email protected]

ossdev07 and others added 2 commits October 9, 2018 16:04
PhantomJS is an unmaintained project removed
optional dependency of phantomJS from mochify.
Updated node version in travis.yml because
earlier version used were very old.

Signed-off-by: ossdev07 <[email protected]>
@ossdev07
Copy link
Author

Ping @jprichardson @az7arul

Can you please have a look at this PR.
It will be a delight to see phantomjs removed from string.js.

@az7arul
Copy link
Collaborator

az7arul commented Dec 7, 2018

Thanks for the PR @ossdev07, will have a look

@ossdev07
Copy link
Author

Hi @az7arul

Did you find the time to review the PR?

@rhenwood-arm
Copy link

Is there any feedback on this patch? I would like to see it landed as phantomjs is unmaintained and not available on AArch64.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants