Skip to content

Cleanup: explicit private modifier and remove dead code#995

Merged
danirabbit merged 2 commits intomainfrom
lenemter/cleanup-7
Feb 19, 2026
Merged

Cleanup: explicit private modifier and remove dead code#995
danirabbit merged 2 commits intomainfrom
lenemter/cleanup-7

Conversation

@lenemter
Copy link
Member

No description provided.

@lenemter lenemter requested a review from a team February 18, 2026 16:06
Copy link
Member

@danirabbit danirabbit 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 except for the one comment :)

@jeremypw
Copy link
Collaborator

This will conflict with #998, which alters a function that this PR removes.

@jeremypw jeremypw added the Status: Blocked Something else needs to be done before this can be fixed label Feb 19, 2026
@jeremypw
Copy link
Collaborator

jeremypw commented Feb 19, 2026

Blocking until conflict resolved and I don't want to merge anything that will require amending translations of the metainfo until after release v8.0.0, unless critical. We can do a quick follow up release.

@danirabbit Will it require re-translation of metainfo if only an <issue></issue> line is added?

@danirabbit
Copy link
Member

danirabbit commented Feb 19, 2026

@jeremypw no <issue> tags are not localized, only what's in <description>

@danirabbit danirabbit removed the Status: Blocked Something else needs to be done before this can be fixed label Feb 19, 2026
@danirabbit danirabbit merged commit 5bfe3a7 into main Feb 19, 2026
4 checks passed
@danirabbit danirabbit deleted the lenemter/cleanup-7 branch February 19, 2026 16:36
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.

3 participants