Skip to content

Format line comments and compiler directives#281

Merged
zaneduffield merged 3 commits into
integrated-application-development:masterfrom
zaneduffield:comment_formatting
Oct 29, 2025
Merged

Format line comments and compiler directives#281
zaneduffield merged 3 commits into
integrated-application-development:masterfrom
zaneduffield:comment_formatting

Conversation

@zaneduffield
Copy link
Copy Markdown
Contributor

Fixes #234
Fixes #237

@zaneduffield zaneduffield requested a review from jgardn3r October 7, 2025 01:39
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 7, 2025

Codecov Report

❌ Patch coverage is 97.95918% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
core/src/rules/comment_contents.rs 97.93% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

Comment thread core/src/rules/comment_contents.rs Outdated
Comment thread CHANGELOG.md Outdated
Copy link
Copy Markdown
Collaborator

@jgardn3r jgardn3r left a comment

Choose a reason for hiding this comment

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

Looking pretty good overall.

Comment thread core/src/rules/comment_contents.rs
Comment thread core/src/rules/comment_contents.rs Outdated
Copy link
Copy Markdown
Collaborator

@jgardn3r jgardn3r left a comment

Choose a reason for hiding this comment

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

Looks good!

@zaneduffield zaneduffield merged commit 33ce34b into integrated-application-development:master Oct 29, 2025
7 checks passed
@zaneduffield zaneduffield deleted the comment_formatting branch October 29, 2025 01:22
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.

Control compiler directive casing Enforce at least one space at start of single-line comments

2 participants