Skip to content
This repository has been archived by the owner on Dec 18, 2018. It is now read-only.

Add compatibility with serverless-offline plugin #6

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

miltador
Copy link

@miltador miltador commented Nov 9, 2016

This PR is based on #1 with my improvements, see my comment at #1.

@coveralls
Copy link

coveralls commented Nov 9, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling e5e71ac on miltador:serverless-offline into 2046429 on silvermine:master.

@jthomerson
Copy link
Member

@miltador I'm sorry for the extreme delay on this PR. It seems by this point the plugin is no longer useful since Lambda introduced support for environment variables and Serverless supported it. I'm curious if you're still using this plugin, and if so, what is the usecase? If it seems like it's still a necessary plugin, I'll look at merging the open PRs and shipping a release. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants