#3071 remove koji.listFaults() method
Closed: Fixed 2 years ago by julian8628. Opened 2 years ago by ktdreyer.

Nothing uses koji.listFaults(). I searched all of GitHub and found no references to 3rd-party code either.

Let's remove this method from the tree.


Looking through the git history, it appeared in the initial code drop, and nothing has ever called this method.

Metadata Update from @tkopecek:
- Custom field Size adjusted to None
- Issue set to the milestone: 1.27
- Issue tagged with: deprecation

2 years ago

@mikem any concerns with directly dropping this? (or rather deprecation warning?)

Well, anything in the main lib needs to have a deprecation period.

My only concern is that we might want something like this functionality (but probably not this implementation) in the future. It's currently tricky to add new faults because an older client will not know about them. It might be nice if they were discoverable.

Metadata Update from @julian8628:
- Issue tagged with: testing-ready

2 years ago

Metadata Update from @jobrauer:
- Issue tagged with: testing-done

2 years ago

Commit 9e6be3f relates to this ticket

Log in to comment on this ticket.

Metadata
Related Pull Requests
  • #3082 Merged 2 years ago