Skip to content
This repository has been archived by the owner on May 17, 2019. It is now read-only.

Add render benchmark to performance API #246

Merged
merged 2 commits into from
Jun 29, 2018
Merged

Add render benchmark to performance API #246

merged 2 commits into from
Jun 29, 2018

Conversation

angus-c
Copy link
Contributor

@angus-c angus-c commented Jun 29, 2018

No description provided.

@angus-c angus-c requested review from lhorie and ganemone June 29, 2018 17:06
@codecov
Copy link

codecov bot commented Jun 29, 2018

Codecov Report

Merging #246 into master will decrease coverage by 0.54%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #246      +/-   ##
=========================================
- Coverage   92.25%   91.7%   -0.55%     
=========================================
  Files          18      18              
  Lines         413     410       -3     
  Branches       79      77       -2     
=========================================
- Hits          381     376       -5     
- Misses         17      18       +1     
- Partials       15      16       +1
Impacted Files Coverage Δ
src/plugins/ssr.js 94.36% <ø> (ø) ⬆️
src/base-app.js 87.21% <0%> (-1.76%) ⬇️
src/server-app.js 100% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 21c60a9...dba8df5. Read the comment docs.

@angus-c angus-c requested a review from rtsao June 29, 2018 17:22
Copy link
Contributor

@KevinGrandon KevinGrandon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know this is only one line, but am a bit worried about slowly adding what might be perceived as runtime bloat to our open source template. Wondering if we could either enable this as a feature flag in the future, or potentially some token that folks can override through the DI system.

@angus-c angus-c merged commit d79c1ca into master Jun 29, 2018
@KevinGrandon KevinGrandon deleted the benchmarks branch July 2, 2018 17:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants