Skip to content

Remove obsolete data type interfaces - #1154

Open
duzda wants to merge 1 commit into
freeipa:mainfrom
duzda:type-cleanup
Open

Remove obsolete data type interfaces#1154
duzda wants to merge 1 commit into
freeipa:mainfrom
duzda:type-cleanup

Conversation

@duzda

@duzda duzda commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

These types are unused, but not caught by knip due to an exception.

Summary by Sourcery

Enhancements:

  • Clean up legacy role, HBAC rule, and sudo rule interfaces that have been superseded by current types to reduce dead code.

These types are unused, but not caught by knip due to an exception.

Signed-off-by: David Hanina <dhanina@redhat.com>

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@veronnicka veronnicka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a no brainer. LGTM

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