Skip to content

Add code fixer for MA0174 (Record should use explicit 'class' keyword)#1070

Merged
meziantou merged 4 commits intomainfrom
copilot/implement-fixer-for-rule-ma0174
Mar 30, 2026
Merged

Add code fixer for MA0174 (Record should use explicit 'class' keyword)#1070
meziantou merged 4 commits intomainfrom
copilot/implement-fixer-for-rule-ma0174

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 30, 2026

  • Create RecordClassDeclarationShouldBeExplicitFixer.cs in src/Meziantou.Analyzer.CodeFixers/Rules/
  • Add code fix tests to RecordClassDeclarationShouldBeExplicitAnalyzerTests.cs
  • Run documentation generator to update README/docs (MA0174 code fix column updated from ❌ to ✔️)
  • Fix merge conflicts in README.md and docs/README.md (MA0173 ✔️ from main, MA0174 ✔️ from this branch, MA0175 ✔️ from previous merge)
  • All tests pass
  • Code review passed (no issues)
  • CodeQL security scan passed (0 alerts)

…rd declarations)

Agent-Logs-Url: https://github.com/meziantou/Meziantou.Analyzer/sessions/45e3346d-f4da-41d5-afc0-596a1487ce72

Co-authored-by: meziantou <509220+meziantou@users.noreply.github.com>
Copilot AI changed the title [WIP] Add fixer for rule MA0174 Add code fixer for MA0174 (Record should use explicit 'class' keyword) Mar 30, 2026
Copilot AI requested a review from meziantou March 30, 2026 15:26
@meziantou meziantou marked this pull request as ready for review March 30, 2026 16:17
@meziantou meziantou enabled auto-merge (squash) March 30, 2026 16:25
@meziantou
Copy link
Copy Markdown
Owner

@copilot There are merge conflict. Can you fix them?

Co-authored-by: meziantou <509220+meziantou@users.noreply.github.com>
auto-merge was automatically disabled March 30, 2026 16:28

Head branch was pushed to by a user without write access

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 30, 2026

@copilot There are merge conflict. Can you fix them?

Merged main into the branch and resolved the conflicts in README.md and docs/README.md — both MA0174 and MA0175 now correctly show ✔️ for code fix (commit 72b99a3).

Copilot AI requested a review from meziantou March 30, 2026 16:30
@meziantou
Copy link
Copy Markdown
Owner

@copilot Can you resolve merge conflict?

…174 code fix preserved)

Co-authored-by: meziantou <509220+meziantou@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 30, 2026

@copilot Can you resolve merge conflict?

Resolved — merged the latest main (which added a code fixer for MA0173) into the branch. MA0173 now shows ✔️ (from main) and MA0174 keeps ✔️ (from this branch) in commit 1445fa9.

This was referenced Apr 13, 2026
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