-
Notifications
You must be signed in to change notification settings - Fork 3
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
major bump @polkadot deps #71
Conversation
@@ -1,27 +1,27 @@ | |||
{ |
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.
The new build process formats the package.json in the same way as @polkadot
does it
# Conflicts: # package.json # packages/types/package.json # yarn.lock
@@ -75,7 +75,6 @@ export function extractEvents(api: ApiPromise, result: ISubmittableResult): IExt | |||
try { | |||
const mod = dispatchError.asModule; | |||
const error = api.registry.findMetaError( | |||
// @ts-ignore |
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.
|
||
**WARN** When the packages are linked in the `js_encointer_service` and yarn install is run, it will create a | ||
`node_modules` folder inside the build folder. Do NOT publish this one. However, with the above sequence, this will be | ||
fine, as `yarn build` cleans the build directory first. |
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.
We don't use that any more, as yarn link does no longer exist with yarn v3
/// This file is almost identical to the one defined in `@polkadot-dev`. It is intended to normalize the publications | ||
/// of `@polkadot` packages. However, it also performs some universally helpful code quality checks. This script does: | ||
/// | ||
/// * build the sources with`CommonJS` and `ESM` output. | ||
/// * normalize the package.json's. | ||
/// * lint the dependencies; make sure every import is indeed declared as a dependency. | ||
/// * makes the output compatible with the new `deno` runtime: https://blog.logrocket.com/what-is-deno/. | ||
|
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.
The heart of the build process, and this is where the meat lies of this PR.
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.
Carte blanche
I feared that we will have to do this soon, as we were two major versions behind. So I thought I will do that endeavour now, as I have time, while there is no stress when potential blockers are discovered. However, this time it was less painful, even though I had to adjust the build-process once again.
I also already tested that the basics work in the encointer-wallet: https://github.com/encointer/encointer-wallet-flutter/tree/cl/major-bump-polkadot
Note: There were three preliminary PRs to this that I did not want to bother you with to speed up the process: