These are chat archives for coala/coala-bears

22nd
Dec 2017
John Vandenberg
@jayvdb
Dec 22 2017 00:24
Ya
John Vandenberg
@jayvdb
Dec 22 2017 00:41
@yash-nisar coala/coala-bears#2191
Monal Shadi
@Monal5031
Dec 22 2017 00:47
What could be the reason that self.run_uut() in tests is not yielding a result? :/ even though that yield is being executed . any help?
John Vandenberg
@jayvdb
Dec 22 2017 01:13
you are 100% confident that yield is emitting a Result ?
Monal Shadi
@Monal5031
Dec 22 2017 01:14
yeah Monal5031/coala-bears#2 here it is
I even put the condition from the Bear inside the test to check if it is correct :sweat:
John Vandenberg
@jayvdb
Dec 22 2017 01:15
what is in body[0] ?
Monal Shadi
@Monal5031
Dec 22 2017 01:15
\n
John Vandenberg
@jayvdb
Dec 22 2017 01:15
always ?
Monal Shadi
@Monal5031
Dec 22 2017 01:16
yeah there is a separate test for that too
John Vandenberg
@jayvdb
Dec 22 2017 01:17
nod.
add a print before the yield to be 100% sure
Monal Shadi
@Monal5031
Dec 22 2017 01:19
okay let me try it once
Monal Shadi
@Monal5031
Dec 22 2017 01:27
It is somehow stripping the \n after body[0], I added 5 \n to be sure :/ Hence not executing the yield I added
Sourabh Pruthi
@sourabh1031
Dec 22 2017 01:30
Hello World
John Vandenberg
@jayvdb
Dec 22 2017 01:38
I think git has a rule of excluding duplicate blank lines
Monal Shadi
@Monal5031
Dec 22 2017 01:40
But while showing out commit message in UI it shows the extra blank lines
John Vandenberg
@jayvdb
Dec 22 2017 01:42
UI = GitHub ?
Monal Shadi
@Monal5031
Dec 22 2017 01:42
yeah
John Vandenberg
@jayvdb
Dec 22 2017 01:42
different git config
Monal Shadi
@Monal5031
Dec 22 2017 01:43
Let see if I can find something in code or externally
will get back to you later on this :sweat_smile:
John Vandenberg
@jayvdb
Dec 22 2017 01:43
nod
Monal Shadi
@Monal5031
Dec 22 2017 01:50
Oh sweet! just experimented a bit and git actually merges all the consecutive \n into two (if more than 2) that too locally
thus making coala/coala-bears#1351 as invalid I guess
John Vandenberg
@jayvdb
Dec 22 2017 03:08
I think it is still valid. It is a setting in git config, and I am guessing we see patches uploaded where that has not happened
Yagyansh Bhatia
@yagyanshbhatia
Dec 22 2017 03:54
how can i launch an extension in vscode??
i am setting up the coala-vs-code, and the documentation in the end just reads "To try it in Visual Studio Code, open ./vscode-client in VS Code and turn to debug view, launch the extension."
John Vandenberg
@jayvdb
Dec 22 2017 03:56
this is the wrong room for plugins ; try the main room
Nitanshu
@nvzard
Dec 22 2017 06:45
@jayvdb can you please take a look at coala/coala#4915 . Just adding the existing code in a try block is leading to test failure(https://ci.appveyor.com/project/coala/coala/build/1.0.10885/job/qvtdji2s1k4iakrf#L285) and I am not able to figure out what the reason is. Can you please take a look ?
John Vandenberg
@jayvdb
Dec 22 2017 06:47
no, sorry
Nitanshu
@nvzard
Dec 22 2017 06:49
Okay no problem :) I'll work harder
Yagyansh Bhatia
@yagyanshbhatia
Dec 22 2017 08:14
.travis.yml
| 14| #··#·The·macOS·build·in·travis·is·really·slow.
| | [NORMAL] YAMLLintBear:
| | comment not indented like content (comments-indentation)
| | *0: Do nothing
| | 1: Open file(s)
| | 2: Add ignore comment
| | Enter number (Ctrl-D to exit):
can someone tell what exactly is this error?
image.png
^^ .travis.yml file
John Vandenberg
@jayvdb
Dec 22 2017 08:44
corobo lmgtfy yamllint comments-indentation
the bot isnt working, but hopefully ^ helps you
Monal Shadi
@Monal5031
Dec 22 2017 13:20
:point_up: December 22, 2017 8:38 AM
@jayvdb I tried that on my other git projects too, same thing happened.
I have no problem sending PR for that btw :P
I have all the necessary changes and tests ready, but they didn't pass the tests locally or on travis
Yagyansh Bhatia
@yagyanshbhatia
Dec 22 2017 13:37
can someone help me with this (i am completely new to yaml)
.travis.yml
| 23| ····-·if·[$TRAVIS_OS_NAME·==·'linux'];·then
| | [NORMAL] PycodestyleBear (E702):
| | E702 multiple statements on one line (semicolon)
Yagyansh Bhatia
@yagyanshbhatia
Dec 22 2017 15:35
i am using coala --files=*.yml --bears=YAMLLintBear --save to check my yml files for errors.
why is it even using pycodestyleBear???
Yagyansh Bhatia
@yagyanshbhatia
Dec 22 2017 15:40
or should i add #noqa to each line to ignore it? is there another way round?
Lasse Schuirmann
@sils
Dec 22 2017 15:40
@yagyanshbhatia you have a .coafile that coala picks up as well?
(check out the -I option for ignoring that)
Yagyansh Bhatia
@yagyanshbhatia
Dec 22 2017 17:25
is there some place where there is bear specific documentation
gitcommitbear is giving Shortlog of HEAD commit isn't in imperative mood! Bad words are 'added'
sorry for asking too much :sob:
John Vandenberg
@jayvdb
Dec 22 2017 19:10
@Monal5031 , could you clarify? You could reproduce the problem of multiple blank lines, or you couldnt ?
Monal Shadi
@Monal5031
Dec 22 2017 19:12
No I couldn't all the various tests I wrote failed not even on coala repos but on other personal repos too
John Vandenberg
@jayvdb
Dec 22 2017 19:29
ah ok.