Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

upgrade to TypeScript 3.5.0 (dev) to fix readonly bug #3275

Merged
merged 2 commits into from
Apr 7, 2019
Merged

upgrade to TypeScript 3.5.0 (dev) to fix readonly bug #3275

merged 2 commits into from
Apr 7, 2019

Conversation

sqs
Copy link
Member

@sqs sqs commented Apr 7, 2019

Fix at microsoft/TypeScript#30734 for an issue with TypeScript 3.4.x and use of readonly T[], which causes webpack to report the following error:

TypeError: Cannot read property 'instantiations' of undefined

Fix at microsoft/TypeScript#30734 for an issue with TypeScript 3.4.x and use of `readonly T[]`, which causes webpack to report the following error:

```
TypeError: Cannot read property 'instantiations' of undefined
```
@sqs sqs requested a review from felixfbecker April 7, 2019 04:04
@sqs sqs requested a review from lguychard as a code owner April 7, 2019 05:10
…ts it)

```
../shared/src/util/memoizeObservable.ts:37:58 - error TS2339: Property 'toString' does not exist on type 'P'.
         const key = resolver ? resolver(params) : params.toString()
```
@codecov
Copy link

codecov bot commented Apr 7, 2019

Codecov Report

Merging #3275 into master will increase coverage by 0.09%.
The diff coverage is 33.33%.

Impacted Files Coverage Δ
client/browser/src/libs/gitlab/api.ts 47.05% <0%> (-2.95%) ⬇️
client/browser/src/shared/backend/diffs.tsx 35.71% <0%> (-2.75%) ⬇️
client/browser/src/libs/bitbucket/api.ts 52.94% <0%> (-3.31%) ⬇️
shared/src/util/memoizeObservable.ts 75% <100%> (ø) ⬆️
cmd/frontend/backend/repos.go 31.68% <0%> (+2.97%) ⬆️
cmd/frontend/backend/repos_vcs.go 29.66% <0%> (+7.62%) ⬆️
cmd/frontend/backend/go_importers.go 59.72% <0%> (+37.5%) ⬆️

@sqs sqs merged commit f845711 into master Apr 7, 2019
@sqs sqs deleted the ts35 branch April 7, 2019 18:55
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