Skip to content

feat: adapt to TVM v10 #3043

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

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

feat: adapt to TVM v10 #3043

wants to merge 5 commits into from

Conversation

skywardboundd
Copy link
Contributor

@skywardboundd skywardboundd commented May 13, 2025

Closes #3042.

Closes #3048.

@skywardboundd skywardboundd linked an issue May 13, 2025 that may be closed by this pull request
Copy link
Member

@anton-trunov anton-trunov left a comment

Choose a reason for hiding this comment

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

The diff should look like this:

image

@novusnota novusnota added this to the Infra: 2025-05 milestone May 13, 2025
@skywardboundd skywardboundd marked this pull request as ready for review May 13, 2025 16:21
@skywardboundd skywardboundd requested a review from a team as a code owner May 13, 2025 16:21
anton-trunov
anton-trunov previously approved these changes May 13, 2025
novusnota
novusnota previously approved these changes May 13, 2025
@skywardboundd skywardboundd marked this pull request as draft May 13, 2025 16:28
@skywardboundd skywardboundd dismissed stale reviews from novusnota and anton-trunov via 471cfb3 May 13, 2025 16:40
@@ -111,20 +111,6 @@ describe("stdlib", () => {
),
);

const addrVar = await contract.getParseVarAddress(
Copy link
Member

Choose a reason for hiding this comment

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

why is this removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

because of ton-blockchain/ton@5fe1526

  • Disable anycast address, allow deploying with "fixed prefix length"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

@novusnota novusnota May 13, 2025

Choose a reason for hiding this comment

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

why is this removed?

With TON version 10 it started to throw exit code 9 almost unconditionally.

Related issue: #3048

@anton-trunov anton-trunov changed the title chore(deps-dev): bump @ton/sandbox from 0.28.0 to 0.30.0 feat: adapt to TVM v10 May 14, 2025
@anton-trunov anton-trunov modified the milestones: Infra: 2025-05, v1.7.0 May 14, 2025
@anton-trunov anton-trunov requested a review from i582 May 14, 2025 07:15
@anton-trunov
Copy link
Member

Let's change the scope of this PR and solve #3048 too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants