chore(tolk): changes for Tolk v1.4.2, rename TON to GRAM#2222
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughDocumentation-wide rename replacing ChangesGRAM-based naming rollout across docs and examples
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@content/standard/tokens/nft/deploy.mdx`:
- Line 209: The `attachGrams` property on line 209 still uses the deprecated
`ton()` helper function while the rest of the example has been updated to use
`grams()`. Replace the `ton("0.005")` call with `grams("0.005")` in the
`attachGrams` property to align with the PR objective of deprecating `ton()` in
favor of `grams()` and maintain consistency with the surrounding updated code.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: a22af8e5-ed28-4b01-91f3-482d2fca7a07
📒 Files selected for processing (15)
content/contract-dev/techniques/gas.mdxcontent/contract-dev/techniques/using-on-chain-libraries.mdxcontent/standard/tokens/nft/deploy.mdxcontent/tolk/basic-syntax.mdxcontent/tolk/examples.mdxcontent/tolk/features/message-sending.mdxcontent/tolk/features/standard-library.mdxcontent/tolk/from-func/stdlib-comparison.mdxcontent/tolk/from-func/tolk-vs-func.mdxcontent/tolk/idioms-conventions.mdxcontent/tolk/syntax/structures-fields.mdxcontent/tolk/syntax/variables.mdxcontent/tolk/types/numbers.mdxcontent/tolk/types/tensors.mdxcontent/tvm/exit-codes.mdx
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@content/contract-dev/techniques/contract-sharding.mdx`:
- Line 67: The inline comment on the line containing the grams() function call
uses outdated terminology, referring to "TON" while the code now uses the
grams() function for consistency with v1.4.2 docs. Update the inline comment to
replace "TON" with "GRAM" or "grams" to match the function name and maintain
terminology consistency throughout the documentation.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 4c2abb72-2901-4c1f-8f5e-398cecd1fafa
📒 Files selected for processing (16)
content/contract-dev/techniques/contract-sharding.mdxcontent/contract-dev/techniques/gas.mdxcontent/contract-dev/techniques/using-on-chain-libraries.mdxcontent/standard/tokens/nft/deploy.mdxcontent/tolk/basic-syntax.mdxcontent/tolk/examples.mdxcontent/tolk/features/message-sending.mdxcontent/tolk/features/standard-library.mdxcontent/tolk/from-func/stdlib-comparison.mdxcontent/tolk/from-func/tolk-vs-func.mdxcontent/tolk/idioms-conventions.mdxcontent/tolk/syntax/structures-fields.mdxcontent/tolk/syntax/variables.mdxcontent/tolk/types/numbers.mdxcontent/tolk/types/tensors.mdxcontent/tvm/exit-codes.mdx
✅ Files skipped from review due to trivial changes (7)
- content/tolk/from-func/stdlib-comparison.mdx
- content/tolk/syntax/variables.mdx
- content/tolk/types/tensors.mdx
- content/tolk/idioms-conventions.mdx
- content/tolk/features/message-sending.mdx
- content/tolk/basic-syntax.mdx
- content/tolk/features/standard-library.mdx
🚧 Files skipped from review as they are similar to previous changes (7)
- content/tolk/from-func/tolk-vs-func.mdx
- content/standard/tokens/nft/deploy.mdx
- content/contract-dev/techniques/using-on-chain-libraries.mdx
- content/contract-dev/techniques/gas.mdx
- content/tolk/syntax/structures-fields.mdx
- content/tolk/types/numbers.mdx
- content/tolk/examples.mdx
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@content/tvm/exit-codes.mdx`:
- Line 44: Update the detailed section for exit code 37 to match the table
terminology. Find the section heading that currently reads "### 37: Not enough
Toncoin" and change it to "### 37: Not enough GRAMs". Also update the error
description text in that same section that references "Not enough Toncoin" to
say "Not enough GRAMs" instead, ensuring consistency with the table entry at
line 44 which already uses the GRAM-based terminology.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 2d16915d-7381-40b4-b13a-d4aeadc6c094
📒 Files selected for processing (16)
content/contract-dev/techniques/contract-sharding.mdxcontent/contract-dev/techniques/gas.mdxcontent/contract-dev/techniques/using-on-chain-libraries.mdxcontent/standard/tokens/nft/deploy.mdxcontent/tolk/basic-syntax.mdxcontent/tolk/examples.mdxcontent/tolk/features/message-sending.mdxcontent/tolk/features/standard-library.mdxcontent/tolk/from-func/stdlib-comparison.mdxcontent/tolk/from-func/tolk-vs-func.mdxcontent/tolk/idioms-conventions.mdxcontent/tolk/syntax/structures-fields.mdxcontent/tolk/syntax/variables.mdxcontent/tolk/types/numbers.mdxcontent/tolk/types/tensors.mdxcontent/tvm/exit-codes.mdx
✅ Files skipped from review due to trivial changes (10)
- content/tolk/syntax/variables.mdx
- content/tolk/idioms-conventions.mdx
- content/tolk/types/tensors.mdx
- content/tolk/basic-syntax.mdx
- content/tolk/types/numbers.mdx
- content/tolk/features/message-sending.mdx
- content/tolk/from-func/stdlib-comparison.mdx
- content/tolk/features/standard-library.mdx
- content/tolk/from-func/tolk-vs-func.mdx
- content/tolk/syntax/structures-fields.mdx
🚧 Files skipped from review as they are similar to previous changes (5)
- content/contract-dev/techniques/contract-sharding.mdx
- content/contract-dev/techniques/gas.mdx
- content/contract-dev/techniques/using-on-chain-libraries.mdx
- content/standard/tokens/nft/deploy.mdx
- content/tolk/examples.mdx
…and grams() in another PR
Since Toncoin is being renamed to GRAM, Tolk compiler was updated here:
For instance,
ton("0.05")is now deprecated overgrams("0.05").This PR reflects those changes in documentation.
NB: These are NOT all required renamings throughout all documentation: they must be performed separately. These are only compiler-related ones. Towards #2209.
Summary by CodeRabbit
grams()(and GRAM-based constants/fields) instead ofton().reserveGramsOnBalance) and updated related validation examples (including freeze due limits).forwardGrams,attachGrams).