Move zip file methods to the NBugZipFile class.

[Stef569]

Hello Teoman,

I refactored the BugReport class. I was looking at the CreateReportZip method and noticed that the if else logic could be replaced by 2 methods:
IsMaxReportingDaysExceeded and IsMaxQueuedReportCountExceeded

then I created a NBugZipFile and moved all the zip logic to that class.

I understand if you don't want to include this as it is a large change...

[soygul]

CreateReportZip was gigantic and needed refactoring for a while. However since nbug has been in production by many, I'm also sure that it works as it should.

Now these changed require proper testing and I don't use Windows anymore so I'm not sure what to do.

[Stef569]

That's ok,

We can include this change later. Maybe in a new release?

Do you want me to close this PR?

[soygul]

To be honest, at this point I can't approve any major changes or even make a new major release as I have no means to test it. I can only read and try to reason about changes as long as they are minor.

Automated testing coverage is also very basic so if you want to continue refactoring big stuff and making major changes, you can keep your own fork of the project.

For instance the biggest user of this library is gitextensions and they keep their own fork: https://github.com/gitextensions/NBug/network