My two thoughts are that we have not had any issues yet due to this and
have been using it for years now and fixing it across the application with
one commit will leave no loose ends, make it clear with one commit why the
change, and making tracking any future issues related easier.
On May 14, 2014 7:11 AM, "Petr Chalupa" <notifications@github.com> wrote:

> In app/models/katello/authorization/activation_key.rb:
>
> >
> > -    def no_tag_verbs
> > -      ActivationKey.list_verbs.keys
> > -    end
> > +      def readable?
>
> A ticket is created. But I not in favor of postponing such things. This is
> how problems begins. It will not take long to convert it for files touched
> by this PR, other locations can be fixed later. Why to keep these files
> misused? Half of the uses could be already fixed.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Katello/katello/pull/4073/files#r12630723>
> .
>