First bot PR changed our table layout

[jerodsanto]

(Thanks so much for building this!)

On first use with an existing contributors list (generated via the command-line tool), the bot sets contributorsPerLine to 7 in our config file. This is undesirable as the 7th column in each row is squished smaller than the others:

PR here

[jakebolam]

Hey @jerodsanto.

Thanks for reporting this! What should be the behaviour?

I took a look at the changelog/transcripts project README (on master) and it appears to be 7 across too. (The CLI has a default of 7, the bot explicit calls this out, you can configure this in your RC file and the bot will respect this).

To avoid confusion we could have the CLI explicitly set this setting too (for consistent behaviour)

[jerodsanto]

lol that's embarrassing I didn't even notice until I saw the bot modify the rc file!

The behavior I'd expect is for the bot to not edit your config at all, but I'd be fine with the CLI also setting that explicitly by default.

[jakebolam]

Yes I agree modifying is a little weird, adding code to not modify in certain cases adds weird complexity to the bot (I also think being explicit in the file is a good idea).

Do you think it would still be confusing even after? all-contributors/cli#149

[jerodsanto]

I think that's Good Enough 👌

[jakebolam]

Thanks @jerodsanto. Any other issues feel free to file. Want to make this really great!