These are chat archives for coala/coala-bears
@jayvdb I'm already done, please review. I'll make the changes if needed
ping @nvzard , can you finish your PR now ?
@jayvdb 0.3.0 was the bad version for dockerfile_lint, and they somehow unreleased that and released the patched version i.e 0.3.1, so there is no bad version
supporting multiple versions which behave differently is not always a good idea.
master is fixed atm
@jayvdb why don't we do a
npm list in CI after all the deps are installed to show the versions
coala/coala-bears#2319 makes it difficult
npm listshow all, or only the versions of the direct dependencies listed in
npm list [name]
I left a comment coala/coala-bears#2319
have you found any solutions for the npm list problem? a custom script is not a good option
let's do that again then? :3
we've uploaded tarballs to github in the past to avoid sf.net
@nvzard , do you have that astyle patch handy ...
mycveit's just a random name :sweat_smile: I will change name and make the PR more understandable by viewing it.
I'll keep that in mind from now on, thanks : )
focus on describing the problem, and the cause, rather than jumping to the solution
MAINTAINERthing in our dockerfiles (because there's something new in Docker v1.3.0)
MAINTAINERtag and the old rule file, i.e. without the newly added rule
npm list -g dockerfile_lint
but I know this shouldn't happen
the MAINTAINER command is deprecated
this must be there in the output^