These are chat archives for coala/coala-bears

2nd
Dec 2016
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 10:53
Hello, I'm a GCI contestant and I'm doing the "advanced asciinema task". So before I actually add it to the GofmtBear page, Is my asciinema good? Here's the link: https://asciinema.org/a/94808
Lasse Schuirmann
@sils
Dec 02 2016 10:55
I like it, maybe show the big patch before applying it so you see what coala wants to change?
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 10:55
also, Is it okay if I make the terminal bigger?
It's the usual 80x25
I wanna make it bigger so It can fit more text
Lasse Schuirmann
@sils
Dec 02 2016 10:56
I think asciinema recommends being <= 100x40 or so so it's easily viewable on mobile devices. Does the GCI instruction text have any guidelines?
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 10:56
actually, no.
Lasse Schuirmann
@sils
Dec 02 2016 10:57
so 100x40 will be good I guess if it doesn't give a warning, it gives a warning if it thinks it's too big
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 10:57
oh okay, didn't know about that. Thanks.
Lasse Schuirmann
@sils
Dec 02 2016 10:58
sure :) how's your GCI experience so far if I might ask?
anything we should improve?
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 10:58
Other than the constant merge conflicts from bad indentation, pretty good.
@sils the tasks descriptions are overkill :P I like the details on the wiki page
Lasse Schuirmann
@sils
Dec 02 2016 10:59
too much?
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 10:59
Nah, it's good
especially for newbies
Lasse Schuirmann
@sils
Dec 02 2016 10:59
well its a lot of work, if it's not needed for most people we gotta rethink that ;)

constant merge conflicts from bad indentation

What's this one?

Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:00
@sils This little PR right here: fossasia/gci16.fossasia.org#24
Lasse Schuirmann
@sils
Dec 02 2016 11:01
ah right, that was the use coala on another project thingy
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:01
actually, it's the task for adding or improving linting to the website
Lasse Schuirmann
@sils
Dec 02 2016 11:03
yeah those problems are always a problem :/ if github just did an inline merge like coala does and maybe show a warning like "this line was automatically merged"...
so maybe we should add a warning to the task or so
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:04
yeah but I think it's a good thing that they made it to do it manually since sometimes automation can do funny things
Lasse Schuirmann
@sils
Dec 02 2016 11:05
true, but there's a third option between fully automating and manually merging: preview changes "as good as possible" and let a human approve
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:05
Yeah, features like that are nice to have.
Lasse Schuirmann
@sils
Dec 02 2016 11:05
which could be in the UI three way merging all conflicting lines and showing a warning somewhere
so someone checks it
dreaming
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:06
dreaming about git automatically solves merge conflicts without a problem :P
Lasse Schuirmann
@sils
Dec 02 2016 11:06
anyway, cool and welcome to the community @yukiisbored :)
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:06
Thanks @sils
Okay just finished a new one, is it okay? https://asciinema.org/a/94811
Lasse Schuirmann
@sils
Dec 02 2016 11:08
oh damn I had another thing I forgot to mention
that warning there isn't too beautiful, add the -I flag to ignore all configs
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:09
oh, didn't see that on --help. Thanks
Lasse Schuirmann
@sils
Dec 02 2016 11:10
we could add the suggestion to the warning, hm?
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:10
Yeah, I think it's a good idea.
Lasse Schuirmann
@sils
Dec 02 2016 11:11

cobot new issue coala Suggest using -I as well at the The default coafile '.coafile' was not found. warning

The user might want to just suppress the warning. We could change the sentence to something like:

You can generate a configuration file with your current options by adding the --save flag or suppress any use of config files with -I.

coala-bot
@coala-bot
Dec 02 2016 11:11
Here you go: coala/coala#3135
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:11
owo
oh wow, didn't know there's a bot that does that
Lasse Schuirmann
@sils
Dec 02 2016 11:12
we built it ourselves ;) lots of bots are acting around coala
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:13
I haven't touched the GitHub API
I wonder what you can do with it
Lasse Schuirmann
@sils
Dec 02 2016 11:13
everything
everything the web UI can do.
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:13
Here's the new one: https://asciinema.org/a/94812
Lasse Schuirmann
@sils
Dec 02 2016 11:14
like it :+1:
Muhammad Kaisar Arkhan
@yukiisbored
Dec 02 2016 11:14
okay gonna add it to my fork, thanks @sils