Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Reformat Apache 2.0 headers (remove #%L things)#28

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
wumpz merged 1 commit intojava-diff-utils:masterfromkoppor:fix-headers
Oct 28, 2018

Conversation

@koppor
Copy link
Contributor

No description provided.

@wumpzwumpz merged commit2c85597 intojava-diff-utils:masterOct 28, 2018
@wumpz
Copy link
Collaborator

After thinking about it. Those %L things are used by Maven license plugin to include and update the LIC text. So what is the reason to remove it?

@koppor
Copy link
ContributorAuthor

koppor commentedOct 29, 2018 via email

Never used the maven license Plugin 😇. In the context of Eclipse Winery,we use IntelliJ's code header functionality or update the headers manually:Each class has the year of the last change.The main reason was consistency to the other classes in the project.I see three options:A) leave as is and update CONTRIBUTING.MDB) Change all headers to maven styleC) Change all headers to a single line SPDX Identifier, which does not holdany year. The copyright holders are contained in the root LICENSE file.Option C makes it easy to maintain, but hard for license scanners. Option Ais IMHO the least WTF, but harder to maintain. Need to check whether themaven plugin can use some other search expression (similar to IntelliJ) sothat option A is also possibleTobias <notifications@github.com> schrieb am Mo., 29. Okt. 2018, 06:50:
After thinking about it. Those %L things are used by Maven license plugin to include and update the LIC text. So what is the reason to remove it? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#28 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABTaflxyqv6NKL2UrxZsifMEWVNYv2hAks5uppc3gaJpZM4X-DTB> .

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@koppor@wumpz

[8]ページ先頭

©2009-2025 Movatter.jp