HR - Timesheet Management

Code review comment for lp:~agilebg/hr-timesheet/imp_hr_attendance_analysis_roundings

Revision history for this message

Pedro Manuel Baeza (pedro.baeza) wrote on 2014-05-12:

Hi, Lorenzo,

Please use correct indentation in if sentence for PEP8 compliance (higher than nested block).

Functionally speaking, it seems good, but maybe you can do:

if (float_end_time - float_start_time) < 0.0000001:

and simplify the condition.

Regards.

review: Needs Fixing (code review)