Feature request: add limit on persistent coverage information about reportgenerator

[danielpalme]

@jatouma:
The setting only applies to the number of files that get parsed.
When generating the HTML report always a maximum of 100 points in the chart gets rendered.

Why does this make sense?! There are two reasons:

• A chart with (many) more points in not readable any more, that's why a fixed limit exists here
• Parsing more files can make sense if you perform many build with little changes. The charts only contain points when there was a change in:
  • number of covered/total lines
  • number of covered/total branches

[danielpalme]

Thanks for your suggestion. Definitely a good idea.

I try to avoid too many command line parameters.
Perhaps a fixed maximum of historic files is a better idea.

Can you please tell me how many historical coverage files you have in your system and how long does it take to parse them all?

[adriangodong]

I've trimmed down the files to 100 and it took 24 seconds to parse. Relevant sample log:
22:24:50 Current Assembly: Common
22:24:52 Reading historic reports
22:25:17 Initializing report builders for report types: Html, PNGChart

Maximum 50 files sounds like a reasonable balance between history and load time.

[danielpalme]

It now limited to 100 files by default. If you want to adjust the limit, you can do this in ReportGenerator.exe.config.

[jatouma]

I have not been able to increase the 100 files limit. I have tried changing the value in the config file to 300, but my history report still shows 100 only. Does ReportGenerator not support increasing that maximum value? From skimming through the code, seems like this should work.