Skip to content

Conversation

@mildaniel
Copy link
Contributor

@mildaniel mildaniel commented Nov 25, 2022

Migrating CI jobs from AppVeyor to GitHub actions.

This PR parallelizes runtime integration tests so that they can complete faster (from ~22 minutes to ~9 minutes).
It also parameterizes the runtime tests so that we can test multiple dependency versions per runtime (e.g. we're testing both npm 8.x and 9.x for Node.js-based runtimes).

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

@mildaniel mildaniel changed the title (TESTING )Add GH action test: Add GH action Nov 29, 2022
@mildaniel mildaniel changed the title test: Add GH action test: Add GH Actions Nov 29, 2022
@mildaniel mildaniel marked this pull request as ready for review November 29, 2022 00:19
@mildaniel mildaniel requested a review from a team as a code owner November 29, 2022 00:19
python-version: ${{ matrix.python }}
- uses: actions/setup-ruby@v1
with:
ruby-version: '^3.1'
Copy link
Contributor

Choose a reason for hiding this comment

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

Lambda supports ruby2.7, so should we setup that version for the tests as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated!

@mndeveci
Copy link
Contributor

This is exciting! 🥳

@mildaniel mildaniel requested a review from mndeveci November 29, 2022 22:21
Copy link
Contributor

@hawflau hawflau left a comment

Choose a reason for hiding this comment

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

approved with a small comment

python:
- "3.7"
- "3.8"
- "3.9"
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: shall we add "3.10" as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think this might result in exceeding the limit of jobs/pr... Might need a separate job that does a smoke test with all versions.

@mildaniel mildaniel merged commit b7e4947 into aws:develop Nov 29, 2022
@mildaniel mildaniel deleted the github-actions-jobs branch November 29, 2022 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants