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

Recognize deployment valid env variables format #8307

Merged
merged 1 commit into from Sep 30, 2020

Conversation

medikoo
Copy link
Contributor

@medikoo medikoo commented Sep 30, 2020

Addresses: #8285 (comment)

Schema was relaxed to recognize all formats as supported for deployment. Improved validation at local invocation where unsupported format is approached

@codecov-commenter
Copy link

Codecov Report

Merging #8307 into master will decrease coverage by 0.04%.
The diff coverage is 22.22%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8307      +/-   ##
==========================================
- Coverage   88.13%   88.09%   -0.05%     
==========================================
  Files         250      250              
  Lines        9371     9371              
==========================================
- Hits         8259     8255       -4     
- Misses       1112     1116       +4     
Impacted Files Coverage Δ
lib/plugins/aws/provider/awsProvider.js 93.21% <ø> (ø)
lib/plugins/aws/invokeLocal/index.js 68.53% <22.22%> (-1.13%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8c4d972...5d48f9f. Read the comment docs.

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

Successfully merging this pull request may close these issues.

None yet

2 participants