I would prefer 2). Keep the instructions there would help reminding people when they are creating PR.
Sijie On Jun 22, 2017 7:43 AM, "Jia Zhai" <[email protected]> wrote: > Seems to me, it is not a big problem to have these lines there. :) > If we decide to move these lines, I would prefer solution 2) or 3). > > On Thu, Jun 22, 2017 at 10:24 PM, Enrico Olivelli <[email protected]> > wrote: > > > Hi, > > with the new templates it is very common to leave these lines in the PR > > > > > > --- > > Be sure to do all of the following to help us incorporate your > > contribution > > quickly and easily: > > > > - [x] Make sure the PR title is formatted like: > > `<Issue #>: Description of pull request` > > `e.g. Issue 123: Description ...` > > - [x] Make sure tests pass via `mvn clean apache-rat:check install > > findbugs:check`. > > - [x] Replace `<Issue #>` in the title with the actual Issue > > number, if there is one. > > > > --- > > > > This is turn will be committed by the merge script > > > > what can we do? > > 1) Remove these lines from the template > > 2) Change the merge script to try to remove them > > 3) Write in the template to manually remove these lines > > > > I would prefer solution 1) > > > > > > -- Enrico > > >
