Skip to content
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

Add support for Serverless Framework's variables. #29

Closed
wants to merge 4 commits into from

Conversation

I-himawari
Copy link

I'm using using sls print command instead of current yaml parser.

@whisller
Copy link
Owner

@I-himawari hey, sorry for VERY late reply! Thank you for pointing out this problem. I fixed that in #33 and will be looking at the env thing you mentioned about in this PR later today/tomorrow.

whisller added a commit that referenced this pull request Dec 15, 2020
@I-himawari in #29 mentioned idea of using `sls print` instead of manually parsing variables.

This PR introduces this functionality.
whisller added a commit that referenced this pull request Dec 15, 2020
@I-himawari in #29 mentioned idea of using `sls print` instead of manually parsing variables.

This PR introduces this functionality.
whisller added a commit that referenced this pull request Dec 15, 2020
@I-himawari suggested improvement in #29 which sets env variables defined in `serverless.yml` file.
This PR introduces this functionality.
@whisller
Copy link
Owner

@I-himawari I will close this issue as improvements you've mentioned are in place. I will be releasing new version shortly.

@whisller whisller closed this Dec 15, 2020
@I-himawari
Copy link
Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants