-
Notifications
You must be signed in to change notification settings - Fork 87
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: update copyright and spdx #1420
Conversation
cd67b5f
to
b13df72
Compare
@tenthirtyam note that we have a feature branch, |
b13df72
to
4e60c2a
Compare
Hey @ksamora - 👋 I'd suggest waiting until the Ryan |
4e60c2a
to
af0a8f2
Compare
@ksamoray It seems the linter error reported here is valid. |
@tenthirtyam after merging branch vpc20-feature-branch, we added multiple files such as nsxt/resource_nsxt_vpc_subnet.go These files already have the broadcom copyright header but it's not the same as the one being added here. |
Sure. I'll review and update these to align. |
af0a8f2
to
29654e3
Compare
@salv-orlando - updated the new files in 29654e3. |
Indeed, we can remove that func. |
29654e3
to
ee77379
Compare
Old copyright still appears in a bunch of files, e.g: ./tools/diffy/diffy.py:# Copyright (c) 2023 VMware, Inc. All Rights Reserved. |
I'll take another pass today. |
ee77379
to
21e2764
Compare
|
- Updates or adds the Broadcom copyright, as needed. - Updates or adds the SPDX-License-Identifier, as needed. - Removes whitespace as noted. Signed-off-by: Ryan Johnson <[email protected]>
21e2764
to
e462f05
Compare
Summary of Pull Request
As discussed with Chip Childers:
Go:
Python:
Type of Pull Request
Please describe:
Related to Existing Issues
Test and Documentation Coverage
For bug fixes or features:
Breaking Changes?