Remove null (breaks stuff)

[sils]

no idea if this is a good change but it does fix #60 for me, feedback/insight from @kriskbx would be appreciated

[coveralls]

Coverage remained the same at 67.368% when pulling aea0ab4 on viper-development:sils/null into 35b8f91 on kriskbx:master.

[coveralls]

Coverage remained the same at 67.368% when pulling aea0ab4 on viper-development:sils/null into 35b8f91 on kriskbx:master.

[coveralls]

Coverage remained the same at 67.368% when pulling aea0ab4 on viper-development:sils/null into 35b8f91 on kriskbx:master.

[naveenkumarsangi]

@sils, this is actually a good and valid change. From the documentation for Time.toHumanReadable, null is an invalid parameter. Reference:

gitlab-time-tracker/src/models/time.js

Lines 104 to 111 in 35b8f91

	/**
	* get human readable
	* @param input
	* @param hoursPerDay
	* @param format
	* @returns {string}
	*/
	static toHumanReadable(input, hoursPerDay = 8, format = time.defaultTimeFormat) {

[kriskbx]

You're totally right. This was unintended, actually the null should have gone in the constructor of the Time object. Thank you for your contribution and for fixing it and sorry for the long delay.