Duplicate definition when using hourly logrotate

[zfLQ2qx2]

I believe I've found a duplication definition issue when doing hourly log rotation:

/etc/cron.hourly/logrotate is defined in both base.pp and hourly.pp, so if you include logrotate::base and logrotate::hourly, then the manifest compile dies. The fix I believe is to remove the definition from one of them, for proof of concept I took it out of base. Is this correct?