Skip to content

chore: update java template license header year#1917

Merged
diegomarquezp merged 4 commits intomasterfrom
diegomarquezp-patch-1
Jan 19, 2024
Merged

chore: update java template license header year#1917
diegomarquezp merged 4 commits intomasterfrom
diegomarquezp-patch-1

Conversation

@diegomarquezp
Copy link
Copy Markdown
Contributor

No description provided.

@diegomarquezp diegomarquezp requested a review from a team January 18, 2024 18:32
@diegomarquezp diegomarquezp requested review from a team January 18, 2024 19:09
@suztomo suztomo changed the title chore: update java template's license header year chore: update java golden file license header year Jan 18, 2024
@suztomo suztomo changed the title chore: update java golden file license header year chore: update java template license header year Jan 18, 2024
JAR_DOWNLOAD_URL = "https://github.com/google/google-java-format/releases/download/google-java-format-{version}/google-java-format-{version}-all-deps.jar"
DEFAULT_FORMAT_VERSION = "1.7"
GOOD_LICENSE = """/*
* Copyright 2023 Google LLC
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we make it a variable so it automatically updates to current year? The tests would automatically start failing in a new year, but I think it could be a better way to force us update the copyright in time. WDYT @suztomo ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a good idea.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a variable to compute the license year.

@diegomarquezp diegomarquezp merged commit 86b9411 into master Jan 19, 2024
@diegomarquezp diegomarquezp deleted the diegomarquezp-patch-1 branch January 19, 2024 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants