-
Notifications
You must be signed in to change notification settings - Fork 240
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
Enhancement/noid/hash service #217
Conversation
@georgehrke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @raghunayyar, @tcitworld and @codeling to be potential reviewers. |
c5ed331
to
701be81
Compare
fixed the missing semicolon |
701be81
to
90000a0
Compare
@@ -136,6 +136,7 @@ module.exports = function(grunt) { | |||
babel: { | |||
options: { | |||
presets: ['es2015'], | |||
plugins: ["array-includes"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, seems like String.prototype.includes
(and same with array) is already available for our requirements : https://kangax.github.io/compat-table/es6/#test-String.prototype.contains
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch!
Will remove it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good but no need for that babel plugin IMHO.
90000a0
to
632c84d
Compare
removed the babel plugin, let's wait for the tests |
implements a hash_service
needed for:
#21
#41