These are chat archives for coala/coala-bears

2nd
Jan 2018
John Vandenberg
@jayvdb
Jan 02 2018 01:54
the bot says no :P
newbazz
@newbazz
Jan 02 2018 02:22
Thanks @Makman2 ;P
John Vandenberg
@jayvdb
Jan 02 2018 05:07
you havent set up coafile checking on https://github.com/newbazz/bomberman , and still have lines which failed like this long line: https://github.com/newbazz/bomberman/blob/master/game/bomb.py#L115
ysahil97
@ysahil97
Jan 02 2018 10:04
My appveyor build has failed saying that 'npm install' requires administrative access https://ci.appveyor.com/project/coala/coala-bears/build/1.0.8468/job/j2tgmg7rlrku451d How should I proceed to this problem?
Mischa Krüger
@Makman2
Jan 02 2018 14:59
@ysahil97 pr link?
ysahil97
@ysahil97
Jan 02 2018 15:00
@Makman2 I have sort of solved this issue, please check out this PR coala/coala-bears#2202
Mischa Krüger
@Makman2
Jan 02 2018 15:01
Ah nvm, just clicked on it via the build page ;)
And yes because you are now installing packages with -g flag
Those rights are required, because then those packages get installed globally
ysahil97
@ysahil97
Jan 02 2018 15:03
@Makman2 So the additional -g flag won't hurt coala-bears?
Mischa Krüger
@Makman2
Jan 02 2018 15:05
Because of that builds fail
ysahil97
@ysahil97
Jan 02 2018 15:07
@Makman2 I still don't understand your statement. Does that mean should I remove the flag or leave it as it is. (Sorry for this. I am still getting used to this environment)
Mischa Krüger
@Makman2
Jan 02 2018 15:10
This flag is causing appveyor to fail
So probably you have to remove it ;)
ysahil97
@ysahil97
Jan 02 2018 15:12
Actually, I myself had added that -g flag because in its absence, the appveyor build failed due to EPERM error caused during "npm install".
Mischa Krüger
@Makman2
Jan 02 2018 15:20
Shouldn't happen without -g
But try a bit around :+1:
ysahil97
@ysahil97
Jan 02 2018 16:11
@Makman2 Can you also review this PR?
Mischa Krüger
@Makman2
Jan 02 2018 16:23
@ysahil97 please make the builds pass first ;)
ysahil97
@ysahil97
Jan 02 2018 16:23
Okay, sure :thumbsup:
ysahil97
@ysahil97
Jan 02 2018 18:00
@Makman2 If you have time today, could you please review coala/coala-bears#2202 . All the builds have now passed.