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
Update: Add option to ignore constructor functions in object-shorthand rule (fixes #4487) #4574
Update: Add option to ignore constructor functions in object-shorthand rule (fixes #4487) #4574
Conversation
078ace6
to
7fbb3b4
Compare
Just wanted to check in on this. |
"type": "array", | ||
"items": [ | ||
{ | ||
"enum": [0, 1, 2] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can now be modified. A change was merged in could of days ago, that doesn't require first parameter in the array to be specified in the schema. Change was made as part of this pull request: #4596
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome - will take a look and fix it. Thanks!
Other then schema, LGTM |
7fbb3b4
to
8c37703
Compare
Updated :) |
"type": "array", | ||
"items": [ | ||
{ | ||
"enum": ["always", "methods", "properties", "never"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this only have "always" and "methods"? (Based on what the documentation says)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, great point
8c37703
to
44d2276
Compare
Looks good, thanks! |
…ctors Update: Add option to ignore constructor functions in object-shorthand rule (fixes #4487)
Not 100% confident about the JSON Schema (first time working with it, so please bear with me) - let me know if anything needs to be changed!