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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build the aws-k8s-1.17 variant by default #1002

Merged
merged 2 commits into from
Jul 23, 2020

Conversation

tjkirch
Copy link
Contributor

@tjkirch tjkirch commented Jul 23, 2020

commit d928c5497606f603d3019a91db6e5b5024fb20e8
Author: Tom Kirchner <tjk@amazon.com>
Date:   Thu Jul 23 13:36:58 2020 -0700

    Change default variant to aws-k8s-1.17

commit 0f4ce06a9a435d4f508afa69cb756180a853c0ed
Author: Tom Kirchner <tjk@amazon.com>
Date:   Thu Jul 23 13:37:06 2020 -0700

    Check in generated sources/models/README.md
    
    A manual change was checked in that included a newline at the end, which
    generated files don't, so this shows up as a difference every time a build is
    run until we check it in.

Testing done:

cargo make built aws-k8s-1.17 OK, and I was still able to build aws-k8s-1.15 with cargo make -e BUILDSYS_VARIANT=aws-k8s-1.15.

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

A manual change was checked in that included a newline at the end, which
generated files don't, so this shows up as a difference every time a build is
run until we check it in.
@tjkirch tjkirch requested a review from bcressey July 23, 2020 21:20
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.

4 participants