[ 
https://issues.apache.org/jira/browse/LUCENE-7013?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Steve Rowe updated LUCENE-7013:
-------------------------------
    Attachment: LUCENE-7013-precommit.patch

Patch with all violations fixed, includes Uwe's last version of the checker 
unchanged.

Other changes:

* fixed the {{unicode-data}} generator to put the package in the right place in 
its generated {{UnicodeProps.java}},
* put a blank line between the license header and the package in all {{.jflex}} 
files, and in the {{.java}} files they generate.

Running precommit and all tests now, will commit if no problems.

> Move license header before package declaration in all *.java files
> ------------------------------------------------------------------
>
>                 Key: LUCENE-7013
>                 URL: https://issues.apache.org/jira/browse/LUCENE-7013
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Shai Erera
>            Assignee: Shai Erera
>            Priority: Minor
>             Fix For: 5.5, trunk
>
>         Attachments: LUCENE-7013-precommit.patch, 
> LUCENE-7013-precommit.patch, LUCENE-7013-precommit.patch, 
> LUCENE-7013-precommit.patch, LUCENE-7013.patch, mvcopyright.py, mvcopyright.py
>
>
> In LUCENE-7012 we committed a change to the IDE templates to place the 
> license header before the package declaration in new Java files.
> I wrote a simple Python script which moves the header before the package 
> declaration. To be on the safe side, if a .java file does not already start 
> with the license header or with {{package org.apache}}, it doesn't modify it 
> and asks for manual intervention.
> It runs quite fast, so I don't mind running and committing one module at a 
> time.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to