Skip to content

Conversation

@rosen-vladimirov
Copy link
Contributor

Currently build of the extension fails as the versions of dependencies and devDependencies are not hardcoded and we receive version of @types/lodash that is incompatible with the TypeScript we are using.
For @types/lodash we have the following dist-tags:

{ latest: '4.14.122',
  'ts2.0': '4.14.50',
  'ts2.1': '4.14.56',
  'ts2.2': '4.14.112',
  'ts2.3': '4.14.112',
  'ts2.4': '4.14.112',
  'ts2.5': '4.14.112',
  'ts2.6': '4.14.121',
  'ts2.7': '4.14.121',
  'ts2.8': '4.14.122',
  'ts2.9': '4.14.122',
  'ts3.0': '4.14.122',
  'ts3.1': '4.14.122',
  'ts3.2': '4.14.122',
  'ts3.3': '4.14.122',
  'ts3.4': '4.14.122' }

In our case we are using TypeScript 2.6.2, so set the version of @types/lodash to 4.14.121.
Also set strict versions for all other dependencies (based on what has been installed in my local node_modules dir).

Currently build of the extension fails as the versions of dependencies and devDependencies are not hardcoded and we receive version of `@types/lodash` that is incompatible with the TypeScript we are using.
For `@types/lodash` we have the following dist-tags:
```
{ latest: '4.14.122',
  'ts2.0': '4.14.50',
  'ts2.1': '4.14.56',
  'ts2.2': '4.14.112',
  'ts2.3': '4.14.112',
  'ts2.4': '4.14.112',
  'ts2.5': '4.14.112',
  'ts2.6': '4.14.121',
  'ts2.7': '4.14.121',
  'ts2.8': '4.14.122',
  'ts2.9': '4.14.122',
  'ts3.0': '4.14.122',
  'ts3.1': '4.14.122',
  'ts3.2': '4.14.122',
  'ts3.3': '4.14.122',
  'ts3.4': '4.14.122' }
```

In our case we are using TypeScript 2.6.2, so set the version of `@types/lodash` to 4.14.121.
Also set strict versions for all other dependencies (based on what has been installed in my local `node_modules` dir).
@cla-bot cla-bot bot added the cla: yes label Mar 5, 2019
@ghost ghost assigned rosen-vladimirov Mar 5, 2019
@ghost ghost added the new PR label Mar 5, 2019
@rosen-vladimirov rosen-vladimirov merged commit ba80dd4 into master Mar 5, 2019
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/fix-build branch March 5, 2019 14:09
@ghost ghost removed the new PR label Mar 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants