Skip to content

Commit

Permalink
FRIDGE-734 make serverless-toolkit work with node v18 (#493)
Browse files Browse the repository at this point in the history
* fix(@twilio-labs/plugin-serverless): add node types to the autogenerated tsconfig

autogenerated tsconfig lacks types at the moment which causes twilio serverless:init
twilio-functions --typescript to fail to start

* Create wise-teachers-clean.md

* docs: add @makserik as a contributor

* fix(create-twilio-function): bump ts from ^4.9.4 to ^5.3.3, fix serverless:init, add plugins testing

* Create ten-cobras-fail.md

* Delete .changeset/wise-teachers-clean.md

* Delete .changeset/ten-cobras-fail.md

* Create six-paws-speak.md
  • Loading branch information
makserik authored Dec 8, 2023
1 parent 83b488c commit d8db768
Show file tree
Hide file tree
Showing 8 changed files with 34 additions and 40 deletions.
54 changes: 19 additions & 35 deletions .all-contributorsrc
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,7 @@
"projectOwner": "dkundel",
"repoType": "github",
"repoHost": "https://github.com",
"files": [
"README.md"
],
"files": ["README.md"],
"imageSize": 100,
"commit": true,
"contributors": [
Expand All @@ -14,79 +12,65 @@
"name": "Dominik Kundel",
"avatar_url": "https://avatars3.githubusercontent.com/u/1505101?v=4",
"profile": "https://dkundel.com",
"contributions": [
"code"
]
"contributions": ["code"]
},
{
"login": "dbbidclips",
"name": "dbbidclips",
"avatar_url": "https://avatars1.githubusercontent.com/u/41997517?v=4",
"profile": "https://github.com/dbbidclips",
"contributions": [
"code",
"bug"
]
"contributions": ["code", "bug"]
},
{
"login": "ShelbyZ",
"name": "Shelby Hagman",
"avatar_url": "https://avatars0.githubusercontent.com/u/1033099?v=4",
"profile": "https://shagman.codes",
"contributions": [
"bug",
"code"
]
"contributions": ["bug", "code"]
},
{
"login": "jsjoeio",
"name": "JavaScript Joe",
"avatar_url": "https://avatars3.githubusercontent.com/u/3806031?v=4",
"profile": "https://joeprevite.com/",
"contributions": [
"bug"
]
"contributions": ["bug"]
},
{
"login": "stefanjudis",
"name": "Stefan Judis",
"avatar_url": "https://avatars3.githubusercontent.com/u/962099?v=4",
"profile": "https://www.stefanjudis.com/",
"contributions": [
"bug",
"code"
]
"contributions": ["bug", "code"]
},
{
"login": "philnash",
"name": "Phil Nash",
"avatar_url": "https://avatars3.githubusercontent.com/u/31462?v=4",
"profile": "https://philna.sh",
"contributions": [
"bug",
"code",
"review"
]
"contributions": ["bug", "code", "review"]
},
{
"login": "childish-sambino",
"name": "childish-sambino",
"avatar_url": "https://avatars0.githubusercontent.com/u/47228322?v=4",
"profile": "https://github.com/childish-sambino",
"contributions": [
"code",
"bug"
]
"contributions": ["code", "bug"]
},
{
"login": "thinkingserious",
"name": "Elmer Thomas",
"avatar_url": "https://avatars0.githubusercontent.com/u/146695?v=4",
"profile": "http://www.ThinkingSerious.com",
"contributions": [
"bug",
"doc"
]
"contributions": ["bug", "doc"]
},
{
"login": "makserik",
"name": "makserik",
"avatar_url": "https://avatars.githubusercontent.com/u/15821542?v=4",
"profile": "https://github.com/makserik",
"contributions": ["code", "doc"]
}
]
],
"commitConvention": "angular"
}

5 changes: 5 additions & 0 deletions .changeset/six-paws-speak.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"create-twilio-function": patch
---

FRIDGE-734 make serverless-toolkit work with node v18
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -72,3 +72,5 @@ dist/
*.twiliodeployinfo

packages/serverless-api/docs/

.idea
2 changes: 1 addition & 1 deletion .nvmrc
Original file line number Diff line number Diff line change
@@ -1 +1 @@
14
18
1 change: 1 addition & 0 deletions docs/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ npm run bootstrap
## Contributing

1. Perform changes. Check out [Working with Workspaces](#working-with-workspaces) for more info
2. If you changed packages/plugins, run `twilio plugins:link <PATH_TO_THE_CHANGED_PACKAGE>` and then test the plugin locally by running `twilio <PLUGIN_NAME> <COMMAND>`
2. Make sure tests pass by running `npm test`
3. Stage the files you changed by running `git add` with the files you changed.
4. If you have a customer facing change make sure to run `npm run changeset` at the root of the project, select what type of version change it is and which packages are impacted and describe the change. Check out ["How we version"](#how-we-version) for more details.
Expand Down
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
"@changesets/cli": "^2.26.0",
"@commitlint/cli": "^9.1.2",
"@commitlint/config-conventional": "^10.0.0",
"@twilio/test-dep": "npm:[email protected]",
"@types/jest": "^29.2.4",
"all-contributors-cli": "^6.1.2",
"commitizen": "^4.2.4",
Expand All @@ -40,8 +41,7 @@
"prettier": "^2.2.1",
"rimraf": "^3.0.2",
"ts-jest": "^28.0.8",
"typescript": "^4.9.4",
"@twilio/test-dep": "npm:[email protected]"
"typescript": "^5.3.3"
},
"lint-staged": {
"*.{js,jsx,ts,tsx}": [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -126,15 +126,17 @@ function createTsconfigFile(pathName) {
JSON.stringify(
{
compilerOptions: {
target: 'es5',
target: 'es2018',
module: 'commonjs',
strict: true,
esModuleInterop: true,
rootDir: 'src',
outDir: 'dist',
skipLibCheck: true,
sourceMap: true,
types: ['node'],
},
exclude: ['node_modules'],
},
null,
2
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ module.exports = {
].replace(/[\^~]/, ''),
twilioRun: pkgJson.dependencies['twilio-run'],
node: '18',
typescript: '^3.8',
typescript: '^5.3.3',
serverlessRuntimeTypes: '^1.1',
copyfiles: '^2.2.0',
};

0 comments on commit d8db768

Please sign in to comment.