Skip to content

chore: Release SAM CLI v0.22 #1413

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.

Sign up for GitHub

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

Jump to bottom
Merged
sanathkr merged 12 commits into from
Sep 17, 2019
Merged

chore: Release SAM CLI v0.22 #1413

sanathkr merged 12 commits into from
Sep 17, 2019

Conversation

@sanathkr
Copy link
Contributor

@sanathkr sanathkr commented Sep 16, 2019

Issue #, if available:

Description of changes:

Checklist:

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

willyg302 and others added 12 commits August 30, 2019 06:36
@willyg302 @sanathkr
fix(generate-event): Remove two remaining cases of double mustache (#…
32cf6be 
…1389)

Everything switched to triple mustache in #824 except for these two
occurrences.
@shortjared @sriram-mv
updates to yamlhelper and related tests (#1388)
a9d69d5
@jfuss @sanathkr
chore: Apply Black automatic code formatting (#1186)
a83aa9e 
Touches all files in the project. Might create merge conflicts for others.
@jfuss
chore: Run black --check on make pr (#1399)
06a91c5
@jfuss
feat: Provide pinned requirements to allow for reproducible builds (#…
2a55807 
…1391)
@stalar @sriram-mv
Require Cors header values to be specified with double quotes. (#1395)
78274a0 
* Require Cors header values to be specified with double quotes.

* Fix broken tests

* Code formatting

* Use startswith and endswith.

* Adjust excpetion message and add unit tests to assert exception is thrown

* format using black
@jfuss @sriram-mv
chore: Handle pytest warnings (#1406)
9355b7b 
* chore: Handle pylint warnings

* fail pytest on warnings
@jfuss @sanathkr
fix(build): Disallow --build-dir to resolve to cwd (#1403)
37d0e39
@jfuss @sanathkr
fix(intrinsics): Attempt to resolve all intrinsics but leave untouche…
aa336bb 
…d in failure (#1410)
@sanathkr @jfuss
chore: Bumping SAM Translator version (#1412)
4b3336d
@sanathkr
fix: Run Integration tests on Windows with AppVeyor (#1269)
feeb1a2
@sanathkr
chore: black format tests (#1415)
f896920
@sanathkr
Copy link
Contributor Author

sanathkr commented Sep 17, 2019

sam package tests passed in Linux. But due to a bug in how we ignore Build related integ tests, the "merge-to-master" AppVeyor run is stuck.

@sriram-mv sriram-mv self-requested a review September 17, 2019 19:57
@sanathkr sanathkr merged commit 1802ed8 into master Sep 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@sriram-mv sriram-mv sriram-mv approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@sanathkr @sriram-mv @willyg302 @shortjared @jfuss @stalar