Update yarn to 3.x (round2) #53
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mhagel
reviewed
Sep 14, 2022
mhagel
approved these changes
Sep 14, 2022
mhagel
added a commit
that referenced
this pull request
Oct 6, 2022
* chore: Upgrade @comjs/stargate and WIP on ecocredit amino converters * chore: Add modules * test: Add amino test * Remove non-essential content and clean up README * Improves for the README * Revert "Remove non-essential content and clean up README" This reverts commit 627f9a9. * Update dependencies, add MsgSend (#39) * feat: add MsgSend to aminomessages.ts * chore: isAminoMsgSend * chore: upgrade @cosmjs/stargate to 0.28.11 and include @cosmjs/proto-signing and @cosmjs/tendermint-rpc in dependencies as spec'd in https://github.com/cosmos/cosmjs/blob/main/packages/stargate/CUSTOM_PROTOBUF_CODECS.md - passing amino test for MsgSend * feat: AminoMsgSend_SendCredits * chore: add MsgSend and MsgCreateClass to customRegistry * wip: test with Secp256k1HdWallet * wip: amino tests * chore: add MsgCancel aminoType * chore: fix cancel_credits type * chore: back to v1 Credits * feat: ecocredit amino converters * chore: update amino dependency * feat: export all converters * feat: if field is falsey, return undefined in toAmino, omit nested (s) * chore: update ecocreditSendConverter with default handling * chore: lint and update basket and marketplace msgs * chore: successful MsgCreateClass test * feat: customMessageTypeRegistry * chore: note for future nested message types in send_amino.ts * chore: turn on marketplace and basket converters * chore: (temporary fix) delete nested msg types until ledger supports them * review: naming changes - customRegistry to regenRegistry * test: add createBasket test * Add patch file for cosmjs fork & fix basket test (#43) * add patch to cosmjs dependency for regen ledger amino support * add patch file * remove console log from patch file * remove patch-package from nested package.json * update package.json * update patch file * add fix for basket.MsgCreate test * chore: MsgPut and MsgTake tests * Update packages/api/src/tx/modules/ecocredit/basket/v1/create_amino.ts Co-authored-by: Mark Hagelberg <mark.hagelberg@gmail.com> Co-authored-by: Mark Hagelberg <9438198+mhagel@users.noreply.github.com> * Update proto files for regen (#44) * Clevinson/clean tests (#45) * udate MsgTake test * update MsgSend test * chore: revert x removal Co-authored-by: tyler <48813565+technicallyty@users.noreply.github.com> * test: amino tests for ecocredit core messages (#46) * test: add amin converter tests for ecocredit core * chore: move ethers to dev dependency * Update packages/api/src/tx/modules/ecocredit/v1/create_batch_amino.ts Co-authored-by: Mark Hagelberg <9438198+mhagel@users.noreply.github.com> * chore: move util file * chore: add file to commit * chore: address review * fix: fromAmino dates use undefined Co-authored-by: Mark Hagelberg <9438198+mhagel@users.noreply.github.com> * fix: msg send typo (#48) * fix: msg send and add test case * chore: re-enable test * fix: MsgCreateBatch fixes and tests (#50) * fix: msg send and add test case * fix: fixes and more tests * chore: remove test disable * Update cosmjs fork to pull from npm (#49) * update version * chore(release): Publish v0.4.0-alpha1.0 * add patch & patch-package to api sub package * chore(release): Publish v0.4.0-alpha1.1 * upgrade yarn and use yarn patch * bump version * chore(release): Publish v0.4.0-alpha1.2 * switch to npm published cosmjs-amino fork * chore(release): Publish v0.4.0-alpha1.3 * reset yarn to 1.x * rm patches * rm patches pt 2 * update yarn version * reset yarn.lock * Mark/marketplace amino (#47) * chore: - MsgSell converter fix and passing test - MsgBuyDirect converter change and (failing) test - MsgCancelSellOrder converter change and (failing) test * chore: omitDefault and longify in marketplace aminos * chore: consolidate util files * chore: fix MsgBuyDirect test * add more test cases * chore: amino test helpers file, fix boolean handling on BuyDirect * Update packages/api/src/tx/modules/ecocredit/converter-utils.ts Co-authored-by: Tyler <48813565+technicallyty@users.noreply.github.com> * chore: use omitDefault for boolean Co-authored-by: Cory Levinson <cjlevinson@gmail.com> Co-authored-by: Tyler <48813565+technicallyty@users.noreply.github.com> * chore: update lint config, lint * Update yarn to 3.x (round2) (#53) * update yarn to stable (3.2.3) * add rimraf * chore: update deps * chore: move to dev deps * update workflow to run CI on all PRs irrespective of branch (#52) * trigger CI * add eslint to dev deps * add eslint to root dev deps * change eslint versino to 7.32.0 * align jest dependency with demo-app * move jest to a root resolution Co-authored-by: tyler <48813565+technicallyty@users.noreply.github.com> * update root & nested jest & ts-jest deps to consistent versions (#54) * chore: disable tests for CI - to avoid unnecessary redwood pollution, experimental release note (#55) * update changelog * chore(release): Publish v0.4.0-alpha1.4 * Revert ledger bug fix (#56) * reverts 849cf0a, adding back in nested $type fields * remove resolution for regen specific cosmjs fork * Revert "reverts 849cf0a, adding back in nested" This reverts commit 81dea9a. * test: add support for local testing Co-authored-by: Mark Hagelberg <mark.hagelberg@gmail.com> * chore: disable integration tests to pass CI * chore: revert yarn release upgrade * chore: v0.4.0 * chore: restore messageTypeRegistry * chore: v0.4.0-alpha1.5 * chore: v0.4.0-alpha1.4 * chore(release): Publish v0.4.0-alpha1.5 * chore(release): Publish v0.4.0 Co-authored-by: Kyle Lawlor-Bagcal <kyle.lawlor@regen.network> Co-authored-by: Mark Hagelberg <9438198+mhagel@users.noreply.github.com> Co-authored-by: Mark Hagelberg <mark.hagelberg@gmail.com> Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> Co-authored-by: tyler <{ID}+{username}@users.noreply.github.com> Co-authored-by: Cory <cjlevinson@gmail.com> Co-authored-by: tyler <48813565+technicallyty@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Second version of this PR which should trigger CI