Skip to content

Conversation

@junxzm1990
Copy link
Contributor

@junxzm1990 junxzm1990 commented Oct 25, 2025

This PR update the language version we used to gate built-in constant __COMPILE_FOR_TESTING__.

@vercel
Copy link

vercel bot commented Oct 25, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
aptos-docs Ready Ready Preview Comment Oct 25, 2025 1:21am

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@junxzm1990 junxzm1990 marked this pull request as ready for review October 25, 2025 01:16
Copilot AI review requested due to automatic review settings October 25, 2025 01:16
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

Updates the documentation to reflect that the __COMPILE_FOR_TESTING__ builtin constant is available since language version 2.2 rather than 2.3, moving it to a separate table to clarify its earlier availability.

  • Reorganized builtin constants documentation to separate constants by their language version availability
  • Moved __COMPILE_FOR_TESTING__ from version 2.3 table to a new version 2.2 table

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


Builtin constants are predefined named values which can be used from anywhere in the code. The following constants are supported:

*since language version 2.2*
Copy link

Copilot AI Oct 25, 2025

Choose a reason for hiding this comment

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

Inconsistent capitalization: this should be 'Since language version 2.2' to match the capitalization style used on line 132.

Suggested change
*since language version 2.2*
*Since language version 2.2*

Copilot uses AI. Check for mistakes.
@junxzm1990 junxzm1990 merged commit c78bc37 into main Oct 25, 2025
4 checks passed
@junxzm1990 junxzm1990 deleted the jun/update-const branch October 25, 2025 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants