DataDog / datadog-cloudformation-macro

CloudFormation Macros by Datadog
Apache License 2.0
14 stars 22 forks source link

Fix env vars when macro is used without extensionLayerVersion set #94

Closed sfirrin closed 1 year ago

sfirrin commented 1 year ago

What does this PR do?

Motivation

https://github.com/DataDog/datadog-cloudformation-macro/issues/92 thanks to @cscetbon

Testing Guidelines

Additional Notes

Types of changes

Check all that apply

codecov-commenter commented 1 year ago

Codecov Report

Merging #94 (c42237d) into master (2e87074) will decrease coverage by 0.03%. The diff coverage is 57.14%.

:exclamation: Current head c42237d differs from pull request most recent head e8de585. Consider uploading reports for the commit e8de585 to get more accurate results

@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
- Coverage   86.46%   86.43%   -0.03%     
==========================================
  Files           8        8              
  Lines         613      612       -1     
  Branches      154      153       -1     
==========================================
- Hits          530      529       -1     
  Misses         72       72              
  Partials       11       11              
Impacted Files Coverage Δ
serverless/src/env.ts 75.77% <57.14%> (-0.15%) :arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more