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

add support for the lib #17

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

add support for the lib #17

wants to merge 3 commits into from

Conversation

bixia
Copy link

@bixia bixia commented Aug 9, 2024

add support for the lib

Copy link
Member

Choose a reason for hiding this comment

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

can you please explain the changes in package.json?

@@ -57,6 +57,7 @@ export class ContractExecutor {
#index = 0;
#verbosity: Verbosity | null = null;
#name: string | null = null;
#libs: Cell | null = null;
Copy link
Member

Choose a reason for hiding this comment

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

btw, what problem does this solve for you?

Copy link
Author

Choose a reason for hiding this comment

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

i need to mock a contract, which is a lib deployed on master chain. and i get the code from master chain. but i find that i need to pass the lib along with the code, so that it can work for me

@anton-trunov anton-trunov requested a review from Gusarich August 17, 2024 04:04
@anton-trunov anton-trunov self-assigned this Aug 17, 2024
"peerDependencies": {
"@ton/core": ">=0.49.2",
"@tact-lang/ton-abi": "^0.0.3",
"@ton/core": "0.56.3",
Copy link
Member

Choose a reason for hiding this comment

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

why this is not a peer dependency anymore?

@anton-trunov
Copy link
Member

@bixia Can you please add a test showing your contribution behaves as expected?

Copy link
Member

@Gusarich Gusarich left a comment

Choose a reason for hiding this comment

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

If we add libraries into the emulator, we should fully support them so that you can mock global masterchain libs, not just those for a single smart contract.

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.

3 participants