refactor: use PascalCase for components name

[clarkdo]

No description provided.

[codecov-io]

Codecov Report

Merging #4396 into dev will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##              dev    #4396   +/-   ##
=======================================
  Coverage   91.29%   91.29%           
=======================================
  Files          56       56           
  Lines        1919     1919           
  Branches      482      482           
=======================================
  Hits         1752     1752           
  Misses        155      155           
  Partials       12       12

Impacted Files	Coverage Δ
packages/vue-renderer/src/renderer.js	100% <100%> (ø)	⬆️

---

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 39b1b8e...975f4a2. Read the comment docs.

[MacZel]

This change introduces an issue. For example, I'm getting an error with <nuxt/> being changed to <Nuxt/> in https://github.com/nuxt/nuxt.js/blob/dev/examples/typescript/layouts/default.vue
If changed back to lowercase it runs smoothly.

Node:

ERROR  [Vue warn]: Invalid component name: "pages/index.vue". Component names can only contain alphanumeric characters and the hyphen, and must start with a letter.

Console:

[Vue warn]: Unknown custom element: <Nuxt> - did you register the component correctly? For recursive components, make sure to provide the "name" option.

found in

---> <Default> at layouts/default.vue
       <Root>

[clarkdo]

@MacZel I'll have a look, thanks for the information.

[clarkdo]

@MacZel I've made some changes, could you have a try on latest dev branch ?

[MacZel]

@clarkdo I will try laetr this day and let you know

[manniL]

@MacZel Vue meta is already a dependency of nuxt itself

[MacZel]

@manniL My bad. Reinstalled the package and now everything is ok.
@clarkdo everything is running ok now

[Jan10]

@clarkdo the with-amp example don't work with <AmpImg>

https://github.com/nuxt/nuxt.js/blob/dev/examples/with-amp/pages/dog.vue
https://www.ampproject.org/docs/reference/components/amp-img

[clarkdo]

@Jan10
Thanks for reminding, I'll change it back.

[dzcpy]

Actually, could you explain why use camel case instead of kebab case? What's the convension?
I can see that they use kebab case in Vue's official documentation, this inconsistancy made me a bit confused.
Is it only a habit thing or there's actually benefit?

[clarkdo]

@dzcpy FYI https://vuejs.org/v2/style-guide/#Component-name-casing-in-templates-strongly-recommended

[dzcpy]

@clarkdo Thanks!