Cleanup GoReturnsBearTest by removing unnecessary information

[sils]

From @sudheesh001 on February 15, 2016 19:48

the GoReturnsBearTest contains the following which needs to be removed

a = ['import "errors"', '',
     'func F() (*MyType, int, error) { return nil, 0, errors.New("foo") }']
b = ['package main', '', 'import "os"', '',
     'func main() {', '\tf, _ := os.Open("foo")', '}']

Put the good_files in the same line like the bad_files

(good_file1,
     good_file2,),
    (bad_file1, bad_file2,))

Also correct GoReturnsBear's run() documentation to show using goreturns instead of using gofmt

difficulty/newcomer

Copied from original issue: coala/coala#1516

[sils]

From @satyamamrev on February 16, 2016 7:39

I want to contribute to this

[sils]

From @sudheesh001 on February 16, 2016 8:24

@satyamamrev You could get started with http://coala.readthedocs.org/en/latest/Users/Install.html#installing-coala-from-source

Here are the contribution guidelines http://coala.readthedocs.org/en/latest/Getting_Involved/README.html

[sils]

From @AbdealiJK on February 21, 2016 11:22

@satyamamrev is doing this in coala/coala#1533

@sils1297 he needs to be added to the org

[sils]

Invitation pending to satyamamrev and hypothesist

2016-02-21 12:22 GMT+01:00 AbdealiJK notifications@github.com:

@satyamamrev https://github.com/satyamamrev is doing this in #1533
coala/coala#1533

@sils1297 https://github.com/sils1297 he needs to be added to the org

—
Reply to this email directly or view it on GitHub
coala/coala#1516 (comment)
.

[sils]

Assigning @hypothesist since he accepted the invitation.

[Adrianzatreanu]

unassigning due to numerous contributions?

[sarthak268]

assign me this

[Adrianzatreanu]

unassigning due to inactivity

[mihai-burduselu]

Assign me this!

[OctavianGuzu]

Assign me this!

[sils]

Both of you are already assigned to another newcomer issues. We only have a limited number of them and they take a lot of time to review etc, consier doing a coala.io/low issue instead if you want more!

[ghost]

I would like to get assigned to this.

[sils]

there you go

[ghost]

I need some clarification

a = ['import "errors"', '',
'func F() (*MyType, int, error) { return nil, 0, errors.New("foo") }']
b = ['package main', '', 'import "os"', '',
'func main() {', '\tf, _ := os.Open("foo")', '}']

Instead of 'a' and 'b' variable names are 'good_file1' and 'good_file2'

(1) Should I remove those lines ?