Skip to content

Add module-level rustdoc to constants submodules#1237

Closed
Copilot wants to merge 2 commits intorefactor/organize-constants-and-builtin-scopingfrom
copilot/sub-pr-1236
Closed

Add module-level rustdoc to constants submodules#1237
Copilot wants to merge 2 commits intorefactor/organize-constants-and-builtin-scopingfrom
copilot/sub-pr-1236

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 9, 2026

Addresses feedback from #1236 requesting documentation for the builtins and identifiers modules introduced during constant reorganization.

Changes

  • Added rustdoc to builtins module clarifying it contains builtin function names (constructors, accessors, operators, utilities)
  • Added rustdoc to identifiers module clarifying it contains reserved keywords and special symbols (self, pattern wildcards)
/// Builtin function names used throughout the AST.
///
/// This module contains string constants for all builtin functions available in the mq language,
/// including constructors, accessors, operators, and utility functions.
pub mod builtins {
    pub const ARRAY: &str = "array";
    // ...
}

/// Reserved identifiers and special symbols used in the language.
///
/// This module contains string constants for reserved keywords and special symbols
/// that have semantic meaning in the mq language, such as `self` and pattern matching wildcards.
pub mod identifiers {
    pub const SELF: &str = "self";
    // ...
}

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Update constant organization and variable scoping Add module-level rustdoc to constants submodules Feb 9, 2026
Copilot AI requested a review from harehare February 9, 2026 12:58
@harehare harehare closed this Feb 9, 2026
@harehare harehare deleted the copilot/sub-pr-1236 branch February 18, 2026 12:41
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.

2 participants