cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From csantanapr <>
Subject [GitHub] cordova-lib pull request: CB-6481 Add unified hooks support for co...
Date Tue, 12 Aug 2014 16:12:51 GMT
Github user csantanapr commented on the pull request:
    @sgrebnov I agree now that plugin hooks are located in a unique place like plugins/mypluginid/src
I can have my node module dependencies isolated without impacting others. and now thinking
about user needing to do npm install, I think the node modules should be a short list so I
can include node_modules directory with the modules already installed with my plugin.
    I just tried var nopt = context.requireCordovaModule('nopt');
    maybe I did a typo before, when it was not working. so far I'm trying to keep the hook
simple without external dependencies.
    on requireCordovaModule I think we should keep it for now, it's a very good backdoor :-)
we can use it during development to come up with what set of things we want to expose.
    For now I need a parser, so adding context.opts.parsers =[{ios:}, {android:}] and array
with parser might be a good thing.
    about context.cmdLine I see it now, I must be blind :-p
    Go ahead and work CordovaProject, let me know if you need help or just feedback.
    I will continue to work on the hook for IBM, and add test cases for in cordova-lib as
I find use cases we want to cover in unit tests

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message