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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

SAM CLI Refresh 馃帀: sam init, Better validation, Tons of bug fixes, and 馃悕 #383

Merged
merged 1 commit into from
May 8, 2018
Merged

SAM CLI Refresh 馃帀: sam init, Better validation, Tons of bug fixes, and 馃悕 #383

merged 1 commit into from
May 8, 2018

Conversation

sanathkr
Copy link
Contributor

@sanathkr sanathkr commented May 8, 2018

Issue #, if available:

Description of changes:
This release is an important milestone - New command to get started with SAM apps, full fidelity SAM Template validation, tons of bug fixes, and greatly improved stability. Oh did I mention, we rewrote the entire CLI from ground up?! Yes, in 馃悕. With this refresh, SAM CLI uses the recently open sourced transform, enabling you leverage any new SAM functionality immediately after release.

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

@sanathkr sanathkr changed the title SAM CLI Refresh 馃帀 SAM CLI Refresh 馃帀: sam init, Better validation, Tons of bug fixes, and 馃悕 May 8, 2018
@sanathkr sanathkr requested a review from jfuss May 8, 2018 20:21
@jfuss
Copy link
Contributor

jfuss commented May 8, 2018

:)

Copy link
Contributor

@jfuss jfuss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#python

@sanathkr sanathkr merged commit 9dcd0bc into aws:develop May 8, 2018
@claudiobizzotto
Copy link

@sanathkr samples folder seems to have been deleted.

@jfuss
Copy link
Contributor

jfuss commented May 8, 2018

@claudiobizzotto Samples/examples can be found over in the Serverless Application Model repo. There are many more over there and are being actively maintained.

I will update the README to reflect this.

@kpx-dev
Copy link

kpx-dev commented May 23, 2018

Hey guys, just wanted to learn why a switch from Go to Python for this repo? I love Python as well, just wanted to learn the reasoning behind it, is it because SAM was written in Python? Thank you.

@sanathkr
Copy link
Contributor Author

@kienpham2000 Precisely. We switched because we wanted to provide full fidelity support for all SAM features in the CLI. This is possible only by bundling the official SAM implementation (written in Python) with the CLI. Hence the switch. The rewrite also changed the architecture of the CLI extensively. It will set us up for a faster pace of innovation :)

@kpx-dev
Copy link

kpx-dev commented May 23, 2018

@sanathkr Thanks for the explanation and awesome work. We love sam!

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

Successfully merging this pull request may close these issues.

None yet

4 participants