-
Notifications
You must be signed in to change notification settings - Fork 357
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
M3-5918: Remove babel
and webpack
following use of tsup
#8495
M3-5918: Remove babel
and webpack
following use of tsup
#8495
Conversation
"axios": "~0.21.4", | ||
"ipaddr.js": "^2.0.0", | ||
"querystring": "^0.2.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
querystring
was not used anywhere so I removed it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bnussman It seems to be used in session.ts
and linodeCreate.reducer.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the api-v4
package.json so packages in manager
should be okay
@@ -39,54 +39,36 @@ | |||
"unpkg": "./lib/iife/index.js", | |||
"dependencies": { | |||
"@linode/validation": "*", | |||
"@types/yup": "^0.29.13", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved this types dependency to devDependencies
@@ -100,5 +82,19 @@ | |||
".{ts,tsx}": [ | |||
"tsc -p tsconfig.json --noEmit true --emitDeclarationOnly false" | |||
] | |||
}, | |||
"jest": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved the jest config into the package.json
to match how Cloud Manager does it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks to be good to go to me, probably just need to merge develop
in to resolve that yarn.lock
conflict
Description
api-v4
andvalidation
withtsup
with backwards compatibility #8484How to test
@linode/api-v4
and@linode/validation
still build and work properly in local development and in our ci pipelines