Skip to content

fix(java): make license header fixing idempotent#1008

Merged
chingor13 merged 7 commits intogoogleapis:masterfrom
chingor13:idempotent-licenses
Mar 31, 2021
Merged

fix(java): make license header fixing idempotent#1008
chingor13 merged 7 commits intogoogleapis:masterfrom
chingor13:idempotent-licenses

Conversation

@chingor13
Copy link
Contributor

Fixes #1007

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Mar 29, 2021
DEFAULT_FORMAT_VERSION = "1.7"
GOOD_LICENSE = """
/*
GOOD_LICENSE = """/*
Copy link

Choose a reason for hiding this comment

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

I notice that GOOD_LICENSE has 2020 hardcoded into it, do we need to update this date over time? how does this work for new libraries.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will update separately.

@chingor13 chingor13 added the automerge Merge the pull request once unit tests and other checks pass. label Mar 31, 2021
@chingor13 chingor13 removed the automerge Merge the pull request once unit tests and other checks pass. label Mar 31, 2021
@chingor13 chingor13 added the automerge Merge the pull request once unit tests and other checks pass. label Mar 31, 2021
@chingor13 chingor13 merged commit f6132d5 into googleapis:master Mar 31, 2021
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Mar 31, 2021
@chingor13 chingor13 deleted the idempotent-licenses branch March 31, 2021 23:39
bcoe pushed a commit that referenced this pull request Oct 13, 2021
bcoe pushed a commit that referenced this pull request Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Java: licence header fixing should be idempotent

3 participants