Codecov Report

Merging #2559 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2559   +/-   ##
=======================================
  Coverage   44.59%   44.59%           
=======================================
  Files          95       95           
  Lines       10007    10007           
=======================================
  Hits         4463     4463           
  Misses       4822     4822           
  Partials      722      722

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 3f42445...e5115c3. Read the comment docs.

@AkihiroSuda i was talking with @dmcgowan yesterday. Maybe we can move all the registrations to the services/server. what do you think?

@crosbymichael per your last comment do you still want this merged or will that change applicability of this PR?

It can be merged

Maybe we can move all the registrations to the services/server.

It seems to decrease configurability

It seems to decrease configurability

We will always have the way to disable the "built-in" plugins via configuration as well as include new plugins and make them the default. Currently we don't have this level of configurability for content but I think we should feel comfortable centralizing the built in registrations without affecting user configurability.