Closed KlaasH closed 6 years ago
Is this live yet?
@mtford90 It is in the sense that it's been pulled into develop
, which we use as the base branch for the repo. Currently we recommend installing and using the library by cloning the repo with Git, so you should be able to access these changes if you're using it that way.
Overview
The Lambda environment comes with the AWS SDK pre-installed, so it wastes space in the bundle to install it ourselves.
Also bumps the version listed in package.json (and installed in development) to the current version pre-installed on Lambda
Notes
devDependencies
won't work because Claudia validates the bundle and it's an error for the package to be missing, but that can be worked around by making at an optional dependency and telling Claudia not to install optional.--no-deps
argument to a few of thedocker-compose
commands in places where thetiler
container is being used but thedatabase
container (on which it depends via thelinks:
parameter) isn't needed.Testing Instructions
node_modules
or runscripts/clean
to clear the existing installation, then runscripts/update
to build the tiler container again. It should still installaws-sdk
and work normally.scripts/publish
orscripts/publish --new
. Export the Lambda function to a zip file and confirm thataws-sdk
is not present in thenode_modules
directory.Resolves #118.