search

CommerceDataService / ITA_Principal_Travel

1 stars 10 forks source link

[WIP] Improve report performance #219

Closed arowla closed 8 years ago

arowla commented 8 years ago

The actual report code could be more well-factored, probably into its own class, but I ain't got time for that. ;)

arowla commented 8 years ago

Hm. I think when I changed the model relationships, it may have broken the region version of the report a bit.... gotta look into that.

codecov-io commented 8 years ago

Current coverage is 63.95%

Merging #219 into develop will increase coverage by 34.01%

@@            develop       #219   diff @@
==========================================
  Files            10         12     +2   
  Lines           521        591    +70   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            156        378   +222   
+ Misses          365        213   -152   
  Partials          0          0

Powered by Codecov. Last updated by a9f1449...f185181