-
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
csvlint: Add csvlint engine #11
Conversation
engines/csvlint/Dockerfile
Outdated
FROM linterhub/image-ruby | ||
LABEL maintainer="linterhub@repometric.com" | ||
RUN apk add --no-cache libcurl && \ | ||
apk add --update make \ |
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.
Why do you need update?
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.
csvlint needs make
, the build fails otherwise.
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.
I doubt you need --update to get make.
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.
Yes, you are right, it worked without --update
. Removing.
"Foo","Bsr","Baz" | ||
|
||
"Biff","Baff","Boff" | ||
"Qux","Teaspoon","Doge" |
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.
Missing new line on EOF.
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.
On purpose, it will emit one more less error if added...
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 sorry, forgot this was a test case :P
Adds missing csvlint using ruby image. Closes #10
gah, I can't access the Docker cloud build. @xferra |
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 for your first contribution, great job! Looks good to me!
Adds missing csvlint using ruby image.
Closes #10