Skip to content

Conversation

@GraysonCox
Copy link
Contributor

@GraysonCox GraysonCox commented Nov 5, 2025

Foreword

#39

This is part of #164.

Note that we had to move src/index.js to the root directory because AWS SAM needs it to be in the same location as the package.json. Previously, the dependencies were not being included in the Lambda function at all, but it wasn't a problem because the old Lambda runtime includes aws-sdk by default. The newer runtime does not include aws-sdk by default.

Changes

  • Update the Node version and AWS Lambda runtime because it is deprecated
  • Update the deployment workflow
    • Allow deploying only dev
    • Use a newer version of setup-node
    • Make it look better

Copy link

@jre4851 jre4851 left a comment

Choose a reason for hiding this comment

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

LGTM

@GraysonCox GraysonCox merged commit 0ff36fb into main Nov 5, 2025
1 of 3 checks passed
@GraysonCox GraysonCox deleted the 39 branch November 5, 2025 19:37
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.

3 participants