Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

update deps 07 05 2021 #1411

Merged
merged 8 commits into from
May 10, 2021

Conversation

thatStrangeGuyThorchain
Copy link
Contributor

@thatStrangeGuyThorchain thatStrangeGuyThorchain commented May 10, 2021

Also includes manual clean-up of yarn.lock

packages not updated:

ncu
Using yarn
Checking /Users/achepurko/Documents/projects/asgardex-electron/package.json
[====================] 104/104 100%

 chart.js           ^2.9.4  →   ^3.2.1     // should be separate PR as it needed to migrate from 2x to 3x version
 cosmos-client     ^0.39.2  →  ^0.42.4     // should be updated at the `xchain-cosmos` first
 react-chartjs-2   ^2.11.1  →   ^3.0.3     // same as `chart.js`
 rxjs               ^6.6.7  →   ^7.0.0     // can not upgrade rxjs to 7x version as there is a conflict with
                                           // @devexperts/rx-utils/dist/observable.utils (see screenshot)
 @storybook/react  ^6.1.21  →   ^6.2.9     // depends on babel loader which version is now not acceptable from CRA
 prettier           ^2.2.1  →   ^2.3.0     // will be updated with next PR as it causes large diff
 ts-loader          ^8.1.0  →   ^9.1.2

ts-loader can not update to 9x version as it does not support webpack<5x which is used by craco

image

- craco/craco
- openapitools/openapi-generator-cli
- antd
- axios
- electron-log
- ethers
- fp-ts
- fp-ts-rxjs
- react-intl
- styled-components
- typescript
deps updated:
- @storybook/addon-actions
- @storybook/addon-controls
- @storybook/addon-knobs
- @storybook/addon-links
- @storybook/addon-viewport
- @storybook/addons
- @storybook/theming
- @testing-library/jest-dom
- @testing-library/react-hooks
- @testing-library/user-event
- @types/fs-extra
- @types/jest
- @types/react
- concurrently
- concurrently
- eslint-config-prettier
- eslint-plugin-prettier
- node-abi
- testcafe
- ts-jest
- webpack-cli
- electron
- electron-builder
@thatStrangeGuyThorchain thatStrangeGuyThorchain changed the title Issue 1408/update deps 07 05 2021 update deps 07 05 2021 May 10, 2021
Copy link
Collaborator

@veado veado left a comment

Choose a reason for hiding this comment

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

From prettier blog about Prettier 2.3. https://prettier.io/blog/2021/05/09/2.3.0.html

This release focuses on fixing long-standing issues in the JavaScript printer. Be warned that, unfortunately, reformatting a project with the new version might result in quite a big diff. If you don’t use ignoreRevsFile to hide such wholesale changes from git blame, it might be about time.

Pls try to revert changes made by latest prettier to follow suggestion for using ignoreRevsFile , Iit might reduce these massive changes in this PR (hopefully)

package.json Outdated Show resolved Hide resolved
@thatStrangeGuyThorchain
Copy link
Contributor Author

From prettier blog about Prettier 2.3. https://prettier.io/blog/2021/05/09/2.3.0.html

This release focuses on fixing long-standing issues in the JavaScript printer. Be warned that, unfortunately, reformatting a project with the new version might result in quite a big diff. If you don’t use ignoreRevsFile to hide such wholesale changes from git blame, it might be about time.

Pls try to revert changes made by latest prettier to follow suggestion for using ignoreRevsFile , Iit might reduce these massive changes in this PR (hopefully)

as discussed at the DMs all prettier-related changes will be moved to the separate PR as github does does not support such feature. it's only for local git blame configs

Copy link
Collaborator

@veado veado left a comment

Choose a reason for hiding this comment

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

Good work!

@thatStrangeGuyThorchain thatStrangeGuyThorchain merged commit c26d38c into develop May 10, 2021
@thatStrangeGuyThorchain thatStrangeGuyThorchain deleted the issue-1408/update-deps-07-05-2021 branch May 10, 2021 13:59
veado added a commit that referenced this pull request Jun 23, 2021
- electron@13, misc. electron* packages
- Node 14.17 (as same as Electron 13 is using)
- Latest typescript@4.3.4
- Latest `concurrently@6.2.0`
- testcafe@1.14.2
- ts-loader@8.3.0 (upgrade to 0.9.x is not possible due issues with
webpack 5 used by craco - still same issue as mentioned in
#1411)
- webpack-cli
veado added a commit that referenced this pull request Jun 24, 2021
## Notes

- `yarn.lock` has been created from scratch (by removing it before running `yarn`)
- Node version update to `14.17` (as same as Electron@13 is using - see https://www.electronjs.org/releases/stable?version=13&page=2#13.0.0)
- `.storybook/main.js` defines `reactDocgen: 'react-docgen'` temporary (just as a workaround as mentioned in styleguidist/react-docgen-typescript#356 (comment))
- Upgrades to latest react-chart | chartjs@3.x needs updates for data and style handling for `PoolDetailsChart*.ts`
- Stick on `cosmos-client@0.39.2` (even a minor update to `@0.39.13` breaks tests - see https://github.com/thorchain/asgardex-electron/pull/1579/checks?check_run_id=2903609210#step:8:27)
- Following packages could not be updated (similar issues we had before in #1411)

```bash
ncu
Checking asgardex-electron/package.json
[====================] 105/105 100%

 @storybook/react  ^6.1.21  →   ^6.3.0 // breaks build of storybook, it seems babel loader version is not compatible with current CRA (as mentioned in #1411)
 @types/node        ^14.17  →   ^15.12 // we stick on Node@14.17 used by Electron@13.x  
 ts-loader          ^8.3.0  →   ^9.2.3 // not compatible w/ webpack <5.0 - see https://github.com/TypeStrong/ts-loader/releases/tag/v9.0.0
 cosmos-client     ^0.39.2  →  ^0.42.5 // `xchain-cosmos` needs to be updated before
 ethers             ^5.1.4  →   ^5.3.1 // `xchain-ethereum` needs to be updated before 
 rxjs               ^6.6.7  →   ^7.1.0 // `fp-ts-rxjs` + `@devexperts/rx-utils/dist/observable.utils` still depends on rxjs@6.x
 ```
luxuereal added a commit to luxuereal/wallet-Asgardex-electron that referenced this pull request Jun 16, 2023
## Notes

- `yarn.lock` has been created from scratch (by removing it before running `yarn`)
- Node version update to `14.17` (as same as Electron@13 is using - see https://www.electronjs.org/releases/stable?version=13&page=2#13.0.0)
- `.storybook/main.js` defines `reactDocgen: 'react-docgen'` temporary (just as a workaround as mentioned in styleguidist/react-docgen-typescript#356 (comment))
- Upgrades to latest react-chart | chartjs@3.x needs updates for data and style handling for `PoolDetailsChart*.ts`
- Stick on `cosmos-client@0.39.2` (even a minor update to `@0.39.13` breaks tests - see https://github.com/thorchain/asgardex-electron/pull/1579/checks?check_run_id=2903609210#step:8:27)
- Following packages could not be updated (similar issues we had before in thorchain/asgardex-electron#1411)

```bash
ncu
Checking asgardex-electron/package.json
[====================] 105/105 100%

 @storybook/react  ^6.1.21  →   ^6.3.0 // breaks build of storybook, it seems babel loader version is not compatible with current CRA (as mentioned in #1411)
 @types/node        ^14.17  →   ^15.12 // we stick on Node@14.17 used by Electron@13.x  
 ts-loader          ^8.3.0  →   ^9.2.3 // not compatible w/ webpack <5.0 - see https://github.com/TypeStrong/ts-loader/releases/tag/v9.0.0
 cosmos-client     ^0.39.2  →  ^0.42.5 // `xchain-cosmos` needs to be updated before
 ethers             ^5.1.4  →   ^5.3.1 // `xchain-ethereum` needs to be updated before 
 rxjs               ^6.6.7  →   ^7.1.0 // `fp-ts-rxjs` + `@devexperts/rx-utils/dist/observable.utils` still depends on rxjs@6.x
 ```
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