Dependencies updates, only import necessary Lodash functions and upda…#146
Dependencies updates, only import necessary Lodash functions and upda…#146paulincai wants to merge 2 commits intometeor-up:masterfrom
Conversation
…te syntax for new library versions.
|
Thanks @paulincai. Did you check if config validation still works? I might need to update Meteor Up to support validation errors from the latest joi version. |
|
Hi @zodern no detailed checks. I implemented the new syntax around Joi, built and used to deploy. I can confirm that deploys work. Ok, I will do some tests. Let's keep this open for now. |
|
@zodern The result: The last array in the result above is the validation result ( validator(config, utils) ). |
…n. Tested for validity of validation result.
|
Ok, I don't really like brackets for single parameter fat arrow functions ((app) => something) but this is how the linter is set so will go with it. @zodern I thing the pull request contains now the updates. I am here in case you would like me do add or change something. |
|
@paulincai do you know what the new minimum node version supported by the updated dependencies are? The docs still say mup-aws-beanstalk works with Node 4, so we will probably need to update that. |
|
I could look at those 1 by 1. There is another option I think of. |
…te syntax for new library versions.