atifaziz / NCrontab

Crontab for .NET
Apache License 2.0
913 stars 139 forks source link

Enable static code & style analysis #112

Closed atifaziz closed 1 year ago

atifaziz commented 1 year ago

This PR enables static code & style analysis and addresses flagged issues. There is one potential binary change where the valueSelector and errorSelector parameters of CrontabSchedule.TryParse or CrontabField.TryParse throw an ArgumentNullException now if either of the arguments are null. This shouldn't really be a breaking change since sending null previously would have not produced any useful behaviour in a program.