-
-
Notifications
You must be signed in to change notification settings - Fork 669
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rule Proposition: Props Casing #92
Comments
@filipalacerda i think better name for this rule is |
@armano2 they can be vue props tho 🤔 |
yes but we don't know if they are props or regular attributes and even if they are props they are still attributes |
Thank you for the proposal! Sounds good to me, but I have some notes.
|
armano2
added a commit
to armano2/eslint-plugin-vue
that referenced
this issue
Aug 2, 2017
filipalacerda
pushed a commit
to filipalacerda/eslint-plugin-vue
that referenced
this issue
Aug 5, 2017
* master: Add rule `vue/require-valid-default-prop`. (vuejs#119) 3.10.0 Update readme to 3.10.0 Chore: remove package-lock.json (vuejs#128) Fix: parserService must exist always (fixes vuejs#125) (vuejs#127) Add rule `require-render-return`. (vuejs#114) 3.9.0 Update package-lock Update: options for `no-duplicate-attributes` (fixes vuejs#112)(vuejs#113) New: add rule `attribute-hyphenation`. (fixes vuejs#92)(vuejs#95) Add namespace check of svg & mathML instead of tag names (vuejs#120)⚠️ Add support for deprecated state in update-rules⚠️ (vuejs#121) Add rules: `no-dupe-keys` and `no-reserved-keys`. (vuejs#88) Chore: Improve tests for name-property-casing & improve documentation (vuejs#115) New: add `require-prop-types` rule (fixes vuejs#19)(vuejs#85) Update: upgrade vue-eslint-parser (fixes vuejs#36, fixes vuejs#56, fixes vuejs#96) (vuejs#116)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Per feedback in #77 (comment):
Rule proposition:
Props Casing
Define a style for the props casing in templates
We (at GitLab) often see inconsistency in the way we provide props through templates:
or
Configuration
'vue/props-casing': ['camelCase'|'kebab-case'|'PascalCase']
camelCase
kebab-case
PascalCase
cc @armano2
The text was updated successfully, but these errors were encountered: