Created the Language checker, first step in the analyze/lint structure (CRAFT-285)
Metadata
Current evaluation
Merged the language checker, establishing the initial analyze/lint structure. Approved by two reviewers after addressing 18 comments, the change adds 168 lines across two files for CRAFT-285.
Suggested action: —
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged the language checker, establishing the initial analyze/lint structure. Approved by two reviewers after addressing 18 comments, the change adds 168 lines across two files for CRAFT-285. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by maintainer facundobatista, this change implements a Language checker as the foundational component of the analyze/lint structure, addressing CRAFT-285. Accepted without review comments. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the language checker implementation, establishing the initial analyze/lint structure. Authored by facundobatista to address CRAFT-285, the change was integrated directly without comments or revisions. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #455 Added an "analyze" command that exposes the linters/attributes results (CRAFT-319). | charmcraft | merged | Merged, adding an analyze command to expose linters and attributes results, resolving CRAFT-319. Approved by two reviewers, the change modifies six files with 444 additions and 4 deletions. | |
| #435 Implemented the Framework checker (CRAFT-316) | charmcraft | merged | Merged implementation of the Framework checker (CRAFT-316). Approved by two reviewers, the commit adds 497 lines across two files. The branch originated from facundobatista:analint-language-checker. | |
| #445 Use linters' analyze from build command (CRAFT-318) | charmcraft | merged | Merged. Integrated linter analysis into the build command to log attributes and update manifest creation. Fixed a missing integration test in checkers/analyze. Approved by reviewers and passed CI checks. | |
| #460 Isolated the linters/checkers, no more shared state between them (CRAFT-383). | charmcraft | merged | Merged refactoring that isolates linters and checkers to eliminate shared state, preventing failures when dependency checkers are ignored or crash. Approved by three reviewers, passed CI, and modified two files. |