PolymerElements / app-storage

Data-pipes and specialized services
61 stars 47 forks source link

from polymer to Polymer in 2.0-preview #106

Closed bartolom closed 7 years ago

bartolom commented 7 years ago

This pull request wants to make this Polymer element consistent with the majority of other Polymer elements in the 2.0-preview branch. The uppercase version "Polymer", "PolymerLabs" and "PolymerElements" are closer to real name of the github project name, like presented in the git URL.

The use of mixed case does not seem to have an effect on bower and JavaScript projects. But other languages like Java are more picky and would benefit from consistency.

I fixed this in the "2.0-preview" branch of this element, because it would be very nice to have this cleaned up and consistent in 2.0 release. I have manually checked 66 elements that have a "2.0-preview" branch. 56 are ok. This element is one of 10 which has these small differences.

I'm mostly concerned with the bower "Main" dependencies, but as I have changed this file already I also fixed this in the devDependencies

This pull request is a manual follow up of PolymerLabs/tedium#47 and PolymerLabs/tedium#48 which try to do this in an automated way, but are stuck.

bartolom commented 7 years ago

I'm closing this pull request, because it is not necessary anymore.

This request was done, before the Polymer team merged their 2.0-preview into the master. I checked the master branch today and the "main" dependencies which I'm mostly concerned about are correctly using "Polymer" with a uppercase "P".

The "devDependencies" are still a bit mixed but that has lower impact for me