Skip to content

Commit

Permalink
Merge pull request #56 from neiman-marcus/update-serverless3-node20
Browse files Browse the repository at this point in the history
Update serverless3 node20
  • Loading branch information
vinothchandrasekarNM authored Apr 11, 2024
2 parents b37d162 + 954d1cc commit 8062992
Show file tree
Hide file tree
Showing 11 changed files with 4,814 additions and 5,739 deletions.
3 changes: 1 addition & 2 deletions .eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,7 @@
"plugin:@typescript-eslint/eslint-recommended",
"plugin:@typescript-eslint/recommended",
"plugin:jest/recommended",
"prettier",
"prettier/@typescript-eslint"
"prettier"
],
"rules": {}
}
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:
- name: setup
uses: actions/setup-node@v1
with:
node-version: '12.x'
node-version: '20.x'

- name: install
run: npm ci
Expand Down
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,9 @@ $ npm install serverless-provisioned-concurrency-autoscaling

Add the plugin to your `serverless.yml`:

Use version 1.9.1 for Serverless 1/ Serverless 2
Use version 2.0.0 for Serverless 3

```yaml
plugins:
- serverless-provisioned-concurrency-autoscaling
Expand Down Expand Up @@ -147,6 +150,7 @@ N/A

- [Clay Danford](mailto:[email protected])
- [Dawid Boissé](mailto:[email protected])
- [Vinoth Kumar Chandrasekar]([email protected])

## Conduct / Contributing / License

Expand Down
2 changes: 1 addition & 1 deletion __tests__/helpers/serverless.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export const serverless: any = {
package: {},
memory: 1024,
timeout: 6,
runtime: 'nodejs12.x',
runtime: 'nodejs20.x',
vpc: {},
versionLogicalId:
'HelloLambdaVersion0Otd4P0e98XajR4keLBshIyWK1yyxFOZK9AJZD7Wo',
Expand Down
6 changes: 3 additions & 3 deletions __tests__/helpers/target.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ export const expectedTarget = {
ServiceNamespace: 'lambda',
RoleARN: {
'Fn::Sub':
'arn:aws:iam::${AWS::AccountId}:role/aws-service-role/lambda.application-autoscaling.amazonaws.com/AWSServiceRoleForApplicationAutoScaling_LambdaConcurrency',
'arn:aws:iam::#{AWS::AccountId}:role/aws-service-role/lambda.application-autoscaling.amazonaws.com/AWSServiceRoleForApplicationAutoScaling_LambdaConcurrency',
},
},
Type: 'AWS::ApplicationAutoScaling::ScalableTarget',
Expand All @@ -29,7 +29,7 @@ export const expectedTargetWithSingleScheduledAction = {
ServiceNamespace: 'lambda',
RoleARN: {
'Fn::Sub':
'arn:aws:iam::${AWS::AccountId}:role/aws-service-role/lambda.application-autoscaling.amazonaws.com/AWSServiceRoleForApplicationAutoScaling_LambdaConcurrency',
'arn:aws:iam::#{AWS::AccountId}:role/aws-service-role/lambda.application-autoscaling.amazonaws.com/AWSServiceRoleForApplicationAutoScaling_LambdaConcurrency',
},
ScheduledActions: [
{
Expand All @@ -56,7 +56,7 @@ export const expectedTargetWithScheduledActions = {
ServiceNamespace: 'lambda',
RoleARN: {
'Fn::Sub':
'arn:aws:iam::${AWS::AccountId}:role/aws-service-role/lambda.application-autoscaling.amazonaws.com/AWSServiceRoleForApplicationAutoScaling_LambdaConcurrency',
'arn:aws:iam::#{AWS::AccountId}:role/aws-service-role/lambda.application-autoscaling.amazonaws.com/AWSServiceRoleForApplicationAutoScaling_LambdaConcurrency',
},
ScheduledActions: [
{
Expand Down
Loading

0 comments on commit 8062992

Please sign in to comment.