Skip to content

Commit 1fa5442

Browse files
authoredMay 23, 2024
update check-style setting for LY corp copyright (line#5678)
Motivation: In the latest developer guide, it is recommended to use `Copyright $today.year LY Corporation`, but the check-style settings have not been adapted accordingly. When this copyright notice is used, a check-style warning occurs. Modifications: - Updated check-style setting for LY corp copyright Result: - Using either LINE copyright or LY copyright will pass the check-style test. <!-- Visit this URL to learn more about how to write a pull request description: https://armeria.dev/community/developer-guide#how-to-write-pull-request-description -->
1 parent 2a0dba1 commit 1fa5442

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed
 

‎settings/checkstyle/checkstyle.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
<!-- Copyright headers -->
3232
<module name="RegexpSingleline">
3333
<property name="id" value="CopyrightHeader"/>
34-
<property name="format" value="^(?:\s|\*)*Copyright\s+[0-9]+\s+LINE Corporation\s*$"/>
34+
<property name="format" value="^(?:\s|\*)*Copyright\s+[0-9]+\s+(LINE|LY) Corporation\s*$"/>
3535
<property name="minimum" value="1"/>
3636
<property name="maximum" value="1"/>
3737
<property name="message" value="missing copyright header"/>

0 commit comments

Comments
 (0)
Please sign in to comment.