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

FRIDGE-734 make serverless-toolkit work with node v18 #493

Merged
merged 9 commits into from
Dec 8, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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',
};