Re: [PATCH 1/1] TESTING: Adding 2 required packages in the documentation file. Replacing MySQL with SQLite in the comment
Thank you Lukas for review. I'll fix spaces issue and re-submit in two new commits. Cheers, Nodiv On Sat, Mar 10, 2018 at 7:55 AM, Lukas Fleischer wrote: > Dear Nodiv, > > On Tue, 06 Mar 2018 at 03:38:08, Nodiv Byzero wrote: >> --- >> TESTING | 2 +- >> schema/Makefile | 1 + >> 2 files changed, 2 insertions(+), 1 deletion(-) >> [...] > > Thank you for working on this! Something must have gone wrong when you > submitted the patch, though. It looks like tabs were replaced by spaces > and thus, the patch cannot be applied as-is. I recommend using > git-send-email(1). > > I would also prefer to have two separate commits (with separate commit > messages) for the two issues that are fixed. > > Thanks! > > Regards, > Lukas
Re: [PATCH 1/1] TESTING: Adding 2 required packages in the documentation file. Replacing MySQL with SQLite in the comment
Dear Nodiv, On Tue, 06 Mar 2018 at 03:38:08, Nodiv Byzero wrote: > --- > TESTING | 2 +- > schema/Makefile | 1 + > 2 files changed, 2 insertions(+), 1 deletion(-) > [...] Thank you for working on this! Something must have gone wrong when you submitted the patch, though. It looks like tabs were replaced by spaces and thus, the patch cannot be applied as-is. I recommend using git-send-email(1). I would also prefer to have two separate commits (with separate commit messages) for the two issues that are fixed. Thanks! Regards, Lukas