These are chat archives for coala/coala-bears

9th
Jul 2018
Harsh Kumar Bhartiya
@harshhx17
Jul 09 2018 05:00
@jayvdb
I read the logs in circleci, can't see how astyle is being installed though...
Unless it is cached
John Vandenberg
@jayvdb
Jul 09 2018 05:00
what does it say about astyle ?
Harsh Kumar Bhartiya
@harshhx17
Jul 09 2018 05:01
It doesn't say anything in the installation part
John Vandenberg
@jayvdb
Jul 09 2018 05:04
it has to.. are you expanding each section ?
or are you looking at an old log
astyle is now mentioned in deps.apt.sh so it must appear in the logs
Harsh Kumar Bhartiya
@harshhx17
Jul 09 2018 05:06
Ohh!
Srry about that, its perfectly fine
I actually only expanded a few sections :sweat:
John Vandenberg
@jayvdb
Jul 09 2018 07:44
@harshhx17, I may need your PR coala/coala-bears#2493 rather urgently
can you rebase and tidy it up
the other option is turning off Circle CI ;-)
we dont need it now
but it doesnt hurt to keep it alive , if it is being maintained ... :P
John Vandenberg
@jayvdb
Jul 09 2018 08:36
@li-boxuan could you rebase on top of coala/coala-bears#2598
Travis should pass. Circle will fail. We can merge and disable circle until it is fixed by @harshhx17 ;)
John Vandenberg
@jayvdb
Jul 09 2018 09:02

@li-boxuan could you rebase on top of coala/coala-bears#2598

I've pushed to your PR again. apologies this means I am the committer. repush without me as committer if you see this and it isnt merged

John Vandenberg
@jayvdb
Jul 09 2018 11:53
And merged. So coala-bears master is now using coala core master
@pareksha your project is now unblocked.
Pareksha Manchanda
@pareksha
Jul 09 2018 11:55
That's a great news, thanks to all developers and @jayvdb for the quick work :)
Cheers!
John Vandenberg
@jayvdb
Jul 09 2018 11:55
There were a few PRs and issues blocked by this. I'll unblock them now
John Vandenberg
@jayvdb
Jul 09 2018 12:06
@aashraybhandar1 coala/coala-bears#2162 should be unblocked now
@RJ722 coala/coala-bears#2199 still busted :-(
(that might be a special case, where the old coala is still being used; need to triple check)
Rahul Jha
@RJ722
Jul 09 2018 12:12
@jayvdb hawk eyeing it.
John Vandenberg
@jayvdb
Jul 09 2018 12:39
@kriti21 , you can now remove the GitCommitBear hack from every .travis.yml
to install coala core and coala-bears do something like coala/coala#5616
once we get one working, should be copy and paste for the rest of them
John Vandenberg
@jayvdb
Jul 09 2018 12:49
ya, that makes sense. thx. ping @paavininanda , you can now finish your PR
Kriti Rohilla
@kriti21
Jul 09 2018 12:50

And merged. So coala-bears master is now using coala core master

Great :+1:

@kriti21 , you can now remove the GitCommitBear hack from every .travis.yml

On it

Paavini Nanda
@paavininanda
Jul 09 2018 12:51
@jayvdb Okay, I'll do
Boxuan Li
@li-boxuan
Jul 09 2018 13:15
And merged. So coala-bears master is now using coala core master
:+1: Sorry was outside, didn't see your message :sweat_smile:
Rahul Jha
@RJ722
Jul 09 2018 13:26
@jayvdb @paavininanda we still need to figure out why that happened ;) Any idea?
John Vandenberg
@jayvdb
Jul 09 2018 13:28
@RJ722 we can enable verbosity to see what is happening maybe
Kriti Rohilla
@kriti21
Jul 09 2018 13:29
@jayvdb Please have a look at coala/projects#600
John Vandenberg
@jayvdb
Jul 09 2018 13:30
But this PR can go in with your suggested change and we create new issues about any other issues
Rahul Jha
@RJ722
Jul 09 2018 13:40
@jayvdb I'm currently on an ssh server (my local machine screwed up :-( ) and I can't grep the output of coala for some reason.
John Vandenberg
@jayvdb
Jul 09 2018 14:10
no worries; we'll get to it another day
John Vandenberg
@jayvdb
Jul 09 2018 15:14
@kriti21 no more prs until you get one working
I reviewed the first one a while ago
Kriti Rohilla
@kriti21
Jul 09 2018 15:25
I am sorry I misunderstood your comment before. Please reply to this https://github.com/coala/projects/pull/600#discussion_r201043381 @jayvdb