fix: Reference variable from loop instead of the one from the validator

aws / aws-lambda-builders

Python library to compile, build & package AWS Lambda functions for several runtimes & framework

Apache License 2.0

334 stars 139 forks source link

fix: Reference variable from loop instead of the one from the validator #558

Closed lucashuy closed 11 months ago

lucashuy commented 11 months ago

Issue #, if available: None

Description of changes: The recent change to update the Python path resolution to find the correct one if pip is required could reference a variable that has not yet been assigned.

This fix updates the two logs to use the path string from the loop instead of the one from the validator output.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

mildaniel commented 11 months ago

Thanks Lucas, I think we need to get this merged before the next SAM CLI release so that we don't break anyone.

Can you raise a LB version bump PR once this change is in?