Skip to content

src(build): port has_lib.js to javascript #872

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

Merged
merged 1 commit into from
Feb 26, 2017

Conversation

zbjornson
Copy link
Collaborator

Tested on Windows. CI covers some sort of linux. Would be good for someone with MacOS and BSD (@hakkiplaten? @ronelliott?) to test.

Ref #754 (fix build on BSD)
Ref #813 (static build)

@zbjornson zbjornson force-pushed the has-lib-js branch 2 times, most recently from 7c0afda to 4358092 Compare February 21, 2017 18:15
@zbjornson zbjornson mentioned this pull request Feb 21, 2017
@LinusU
Copy link
Collaborator

LinusU commented Feb 22, 2017

Amazing work!

Would you mind adding this file to the standard list here package.json#L27 and update the file to follow standard style? (standard --fix util/has_lib.js) ❤️

@zbjornson
Copy link
Collaborator Author

My poor semicolons! 😜 Change made.

@LinusU
Copy link
Collaborator

LinusU commented Feb 26, 2017

Hehe, thank you 🙌

@LinusU LinusU merged commit 610e3ad into Automattic:master Feb 26, 2017
@zbjornson zbjornson deleted the has-lib-js branch February 27, 2017 00:37
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.

2 participants