Skip to content

feat: sort diagnostics#22

Merged
kdkasad merged 1 commit intomasterfrom
order
Jun 28, 2025
Merged

feat: sort diagnostics#22
kdkasad merged 1 commit intomasterfrom
order

Conversation

@kdkasad
Copy link
Owner

@kdkasad kdkasad commented Jun 28, 2025

Sorts printed diagnostics based on a CLI option, either by source line number or by code standard rule.

Sorts printed diagnostics based on a CLI option, either by source line
number or by code standard rule.
@kdkasad kdkasad requested a review from Copilot June 28, 2025 05:22
@kdkasad kdkasad self-assigned this Jun 28, 2025
@kdkasad kdkasad added the enhancement New feature or request label Jun 28, 2025
Copy link

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 pull request introduces functionality to sort printed diagnostics based on a CLI option, either by source line number or by code standard rule. It also refactors the import structure and consolidates diagnostic collection using a functional style.

  • Consolidated standard library and external crate imports into nested structures.
  • Added a new CLI argument to specify the sort order.
  • Refactored rule diagnostics aggregation with a flat_map and added a match block to handle sorting.

@github-actions
Copy link

Test Results

41 tests   41 ✅  0s ⏱️
 3 suites   0 💤
 1 files     0 ❌

Results for commit e53ae8a.

@kdkasad kdkasad merged commit 180f691 into master Jun 28, 2025
17 checks passed
@kdkasad kdkasad deleted the order branch June 28, 2025 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants