Skip to content
This repository was archived by the owner on Jun 17, 2022. It is now read-only.

Upgrade TypeScript #148

Merged
merged 4 commits into from
Aug 12, 2020
Merged

Upgrade TypeScript #148

merged 4 commits into from
Aug 12, 2020

Conversation

Hinton
Copy link
Member

@Hinton Hinton commented Aug 12, 2020

TypeScript 3.6+ is required to access newer versions of the navigator. apis.

  • TypeScript updated to 3.6.5 which is the minimum minor version required.
  • TSLint updated to latest, however TSlint is now deprecated so we probably want to switch to ESlint and Prettier sometime in the future.
  • @types/node updated to 10.17.28

@kspearrin
Copy link
Member

We'll need to have a follow up PR in all repos that match this upgrade.

package.json Outdated
@@ -30,7 +30,7 @@
"@types/jasmine": "^3.3.12",
"@types/lowdb": "^1.0.5",
"@types/lunr": "^2.3.3",
"@types/node": "^10.9.4",
"@types/node": "^12.12.54",
Copy link
Member

Choose a reason for hiding this comment

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

We still use node 10 for building our apps. Is this necessary?

Copy link
Member Author

Choose a reason for hiding this comment

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

We get compile errors otherwise, I can try using a newer version of 10.

Copy link
Member Author

Choose a reason for hiding this comment

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

10.17.28 seems to work

@kspearrin kspearrin merged commit e516692 into bitwarden:master Aug 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants