Skip to content

Conversation

@svc-excavator-bot
Copy link
Collaborator

@svc-excavator-bot svc-excavator-bot commented Feb 10, 2020

excavator is a bot for automating changes across repositories.

Changes produced by the roomba/latest-baseline-oss check.

Release Notes

3.4.0

Type Description Link
Improvement Enforcing copyright based on the first file (lexicographically) found in .baseline/copyright. #1217
Improvement For repos that use snapshot-style testing, ./gradlew test -Drecreate=true will ensure the "recreate" system property is passed through to Java correctly. #1220

3.4.1

Type Description Link
Fix Auto-fixing license headers will attempt to preserve the existing year of creation. #1227

3.4.2

Type Description Link
Fix Copyright header enforcement now includes any leading and trailing whitespace on lines #1228

To enable or disable this check, please contact the maintainers of Excavator.

* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
Copy link
Contributor

Choose a reason for hiding this comment

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

Our thingy should be smart enough to preserve this somehow.

Copy link
Contributor

Choose a reason for hiding this comment

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

I've just made a PR to separate this out to a new block: #1232

@svc-excavator-bot svc-excavator-bot force-pushed the roomba/latest-baseline-oss branch from 1abcc36 to 3831ba4 Compare February 10, 2020 23:56
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
* http://www.apache.org/licenses/LICENSE-2.0
Copy link
Contributor

@iamdanfox iamdanfox Feb 11, 2020

Choose a reason for hiding this comment

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

Argh I think I'm too enthusiastically trimming things- this is supposed to be indented.

(c) Copyright ${today.year} Palantir Technologies Inc. All rights reserved.

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

    http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.

Fix here #1228

* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Still gotta fix this bit 👎

* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://url-to-some-license
Copy link
Contributor

Choose a reason for hiding this comment

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

great one over here :D

@@ -1,5 +1,5 @@
/*
* Copyright 2018 The Error Prone Authors.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure this is fine to change

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants