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

Update readme #13

Merged
merged 1 commit into from Feb 16, 2019
Merged

Conversation

maxnordlund
Copy link
Contributor

This does two things, and if you don't like one of the changes they can easily be rebased away.

1, It reenables the most of the badges.
2, It updates the wording regarding phantomjs support for Function.prototype.bind

The commit messages contains more info and links.

Enable badges

Gemnasium has been closed since it was acquired by GitLab, thus it is
removed. The build and coverage links don't work, but that might be
caused by an configuration issue, so they are left as disabled for now.

https://docs.gitlab.com/ee/user/project/import/gemnasium.html

Update wording regarding phantomjs support

While old version does not support Function.prototype.bind, the latest
(2.1.1) does. Phantomjs is also no longer in development, so it's safe
to assume it will stay the same. Even if it becomes active, it most
likely won't go back to an older, unsupporting, version of QT/WebKit.

http://phantomjs.org/release-2.1.html
ariya/phantomjs#15344
Copy link
Collaborator

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Thanks! just a few changes.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@maxnordlund
Copy link
Contributor Author

I've applied your suggestions now.

@ljharb ljharb force-pushed the maxnordlund-update-readme branch 2 times, most recently from 4f1ce73 to 503cb12 Compare February 16, 2019 22:18
Copy link
Collaborator

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Thanks!

@ljharb ljharb merged commit 503cb12 into Raynos:master Feb 16, 2019
@maxnordlund maxnordlund deleted the maxnordlund-update-readme branch February 16, 2019 22:31
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.

None yet

2 participants