optionally allow asset publishing

ahmedkhattab commented 2 years ago

flag enable_asset_publishing is used to control whether to add an optional step to publish cdk assets to s3, this is useful if the target code repository makes use of assets (for example lambda function code as an asset)

Can be further improved by:

looping over assets manifests (for now the 3 asset manifests for 3 environments are hardcoded)
externalising this logic to a separate script/construct to be reusable in any pipeline construct

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Fatema commented 2 years ago

This looks good! please pull latest changes then will merge this to the main branch :)

ahmedkhattab commented 2 years ago

changes pulled for this pr and the other one with the policies change Thanks you!

aws-samples / sagemaker-custom-project-templates

optionally allow asset publishing #38