Skip to content
This repository was archived by the owner on Jul 30, 2025. It is now read-only.

Add utf-8 comment support#971

Merged
junxzm1990 merged 2 commits intomainfrom
jun/utf8-in-comment
Jun 6, 2025
Merged

Add utf-8 comment support#971
junxzm1990 merged 2 commits intomainfrom
jun/utf8-in-comment

Conversation

@junxzm1990
Copy link
Contributor

Description

Starting Aptos CLI 7.4.0, UTF-8 characters are allowed in comments of Move source code. This PR add a note about the change in the Move book.

Checklist

  • If any existing pages were renamed or removed:
    • Were redirects added to next.config.mjs?
    • Did you update any relative links that pointed to the renamed / removed pages?
  • Do all Lints pass?
    • Have you ran pnpm fmt?
    • Have you ran pnpm lint?

Copilot AI review requested due to automatic review settings June 6, 2025 18:17
@junxzm1990 junxzm1990 requested a review from a team as a code owner June 6, 2025 18:17
@vercel
Copy link

vercel bot commented Jun 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
developer-docs-nextra ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 6, 2025 7:33pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Move book to note that the Aptos CLI now supports UTF-8 characters in comments.

  • Adds a new note about UTF-8 support in comments starting with CLI version 7.4.0
Comments suppressed due to low confidence (1)

apps/nextra/pages/en/build/smart-contracts/book/coding-conventions.mdx:70

  • [nitpick] Consider rephrasing for consistency and clarity, e.g., 'As of Aptos CLI v7.4.0, comments can include UTF-8 characters.' You might also link to the release notes for reference.
- Starting Aptos CLI 7.4.0, UTF-8 characters are allowed in comments.

@junxzm1990 junxzm1990 enabled auto-merge (squash) June 6, 2025 18:29
@junxzm1990 junxzm1990 merged commit aa29484 into main Jun 6, 2025
4 of 5 checks passed
@junxzm1990 junxzm1990 deleted the jun/utf8-in-comment branch June 6, 2025 19:33
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.

4 participants