Skip to content

docs: rewrite comments documentation#9591

Merged
orizi merged 1 commit intostarkware-libs:mainfrom
Evelina331:comments_docs
Feb 3, 2026
Merged

docs: rewrite comments documentation#9591
orizi merged 1 commit intostarkware-libs:mainfrom
Evelina331:comments_docs

Conversation

@Evelina331
Copy link
Contributor

Summary

Rewrote the comments documentation to include doc comments (///) and module-level comments (//!), providing a complete overview of commenting styles.


Type of change

Please check one:

  • Bug fix (fixes incorrect behavior)
  • New feature
  • Performance improvement
  • Documentation change with concrete technical impact
  • Style, wording, formatting, or typo-only change

⚠️ Note:
To keep maintainer workload sustainable, we generally do not accept PRs that
are only minor wording, grammar, formatting, or style changes.
Such PRs may be closed without detailed review.


Why is this change needed?

The previous documentation only mentioned line comments (//) and ignored documentation comments, which are essential for documenting APIs.


What was the behavior or documentation before?

The file contained a single sentence about line comments.


What is the behavior or documentation after?

The file now details all three comment types with examples showing how to use them for code explanation and documentation generation.


Related issue or discussion (if any)


Additional context

@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Collaborator

@orizi orizi left a comment

Choose a reason for hiding this comment

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

:lgtm:

@orizi reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @Evelina331).

@orizi orizi enabled auto-merge February 3, 2026 09:20
@orizi orizi added this pull request to the merge queue Feb 3, 2026
Merged via the queue into starkware-libs:main with commit 6b750fa Feb 3, 2026
52 checks passed
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