useless parsing of tasks

[Rocco83]

Hi,

given the code, if the latest job is found with "no new packages" is found, then if checkrepos is "on" all of the tasks are parsed up to find out a task that match the channel that we need for.
We are here at the code

katello-cvmanager/cvmanager

Line 386 in 64778cf

if @options[:checkrepos]

When one repository is found, further check are done

katello-cvmanager/cvmanager

Line 413 in 64778cf

if tasker['input']['repository']['id'] == repo['id']

A deeper check start

katello-cvmanager/cvmanager

Line 200 in 64778cf

def checktask(task, last_date)

When the task is from the past, the code is just skipping this task

katello-cvmanager/cvmanager

Line 205 in 64778cf

puts_verbose "#{task['humanized']['output']} This past task will NOT trigger a Publish."

Wondering if in this section of code, if the task is from the paste, we can skip the remaining tasks.

katello-cvmanager/cvmanager

Line 415 in 64778cf

if ( checktask(tasker, cv_last_published) )

Looking at the code , my understanding is that if a "past task" is found, since the tasks are sorted by ended_at, we can be sure that no other tasks will be found afterwards with packages that are missing.

Any comment?