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

Bump up sqs worker templates to Node 16 #862

Merged
merged 7 commits into from
May 13, 2022
Merged

Conversation

samchungy
Copy link
Contributor

@samchungy samchungy commented May 11, 2022

It's finally here......

@changeset-bot
Copy link

changeset-bot bot commented May 11, 2022

🦋 Changeset detected

Latest commit: 5b66395

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
skuba Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@72636c
Copy link
Member

72636c commented May 11, 2022

Truly a momentous occasion

architecture: arm64
stackName: ${self:service}
stage: ${env:ENVIRONMENT}
versionFunctions: true
Copy link
Contributor Author

@samchungy samchungy May 11, 2022

Choose a reason for hiding this comment

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

@72636c
Copy link
Member

72636c commented May 11, 2022

Can we upgrade the CDK template too?

@samchungy
Copy link
Contributor Author

samchungy commented May 11, 2022

Can we upgrade the CDK template too?

Awaiting this to be released :( aws/aws-cdk#20261. Guess this PR can wait a little bit more

@72636c 72636c added the dino:snooze Snooze in Review Dino label May 12, 2022
@@ -22,11 +26,5 @@
"test": "skuba test",
"test:ci": "skuba test --coverage",
"test:watch": "skuba test --watch"
},
"skuba": {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Aligning with other templates

@72636c 72636c marked this pull request as ready for review May 13, 2022 01:37
@72636c 72636c requested review from a team as code owners May 13, 2022 01:37
Copy link
Member

@72636c 72636c left a comment

Choose a reason for hiding this comment

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

Great stuff!

@samchungy samchungy changed the title Bump up sqs worker template to Node 16 Bump up sqs worker templates to Node 16 May 13, 2022
@samchungy samchungy merged commit 0e187b8 into master May 13, 2022
@samchungy samchungy deleted the sqs-worker-node-16 branch May 13, 2022 01:49
@seek-oss-ci seek-oss-ci mentioned this pull request May 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dino:snooze Snooze in Review Dino
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants