Skip to content

Improve ConsistencyCheck code (and one minor other code place)#13795

Merged
subhramit merged 8 commits intomainfrom
refine
Sep 1, 2025
Merged

Improve ConsistencyCheck code (and one minor other code place)#13795
subhramit merged 8 commits intomainfrom
refine

Conversation

@koppor
Copy link
Copy Markdown
Member

@koppor koppor commented Aug 31, 2025

Follow-up to #13584.

Fixes consistency check to handle unknown entry types again.

(Also to #12433)

Minor code refactorings.

Steps to test

Run consistency check

Mandatory checks

@koppor koppor added dev: code-quality Issues related to code or architecture decisions component: consistency-check labels Aug 31, 2025
@koppor koppor changed the title Remove strange variable Improve ConsitencyCheck code (and one minor other code place) Aug 31, 2025
@koppor koppor added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Aug 31, 2025
@subhramit subhramit changed the title Improve ConsitencyCheck code (and one minor other code place) Improve ConsistencyCheck code (and one minor other code place) Sep 1, 2025
…ckDialogViewModel.java

Co-authored-by: Subhramit Basu <subhramit.bb@live.in>
@trag-bot
Copy link
Copy Markdown

trag-bot Bot commented Sep 1, 2025

@trag-bot didn't find any issues in the code! ✅✨

@subhramit subhramit enabled auto-merge September 1, 2025 20:32
@subhramit subhramit added this pull request to the merge queue Sep 1, 2025
Merged via the queue into main with commit fdc8687 Sep 1, 2025
38 checks passed
@subhramit subhramit deleted the refine branch September 1, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: consistency-check dev: code-quality Issues related to code or architecture decisions status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants