-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add additional valid parameters to linting rules #6
Conversation
@awmcclain - LMK if you want any changes here! |
Also - I'm assuming you can squash merge, but if not I can make this a single clean commit. |
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.
Thank you thank you!!!!
app/linter/ink-linter.js
Outdated
linaHands: ['checkwatch'] | ||
cutCamera: [ | ||
'linapov', | ||
'LinaFrontClose', |
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 these be all lower for the linter?
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.
@awmcclain - fixed!
yea, they should. I had originally made a change that mapped through the
parameters and also put them lower-case, but decided to pull it out and
didn't update these. I'll test and put a new commit up in the morning.
…On Mon, Jan 14, 2019 at 8:59 PM Andrew McClain ***@***.***> wrote:
***@***.**** commented on this pull request.
Thank you thank you!!!!
------------------------------
In app/linter/ink-linter.js
<#6 (comment)>:
> @@ -57,8 +124,28 @@ let storyTags = {
linter: hasStoryTagError,
needsParam: true,
validParams: {
- cutCamera: ['linapov', 'paxleft', 'paxright', 'dashcam', 'hoodcam', 'linafront', 'city'],
- linaHands: ['checkwatch']
+ cutCamera: [
+ 'linapov',
+ 'LinaFrontClose',
Should these be all lower for the linter?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#6 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAqWg7DTjbGZ2VL4lfX0TPM7HQaKZp6Nks5vDWA6gaJpZM4aAM_p>
.
|
No description provided.