Skip to content
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

Merged
merged 1 commit into from
Jan 20, 2025
Merged

Conversation

tenthirtyam
Copy link
Collaborator

@tenthirtyam tenthirtyam commented Oct 12, 2024

Summary of Pull Request

As discussed with Chip Childers:

  • Updates or adds the Broadcom copyright, as needed.
  • Updates or adds the SPDX-License-Identifier, as needed.
  • Removes whitespace as noted.

Go:

// © Broadcom. All Rights Reserved.
// The term "Broadcom" refers to Broadcom Inc. and/or its subsidiaries.
// SPDX-License-Identifier: MPL-2.0

Python:

# © Broadcom. All Rights Reserved.
# The term "Broadcom" refers to Broadcom Inc. and/or its subsidiaries.
# SPDX-License-Identifier: MPL-2.0

Type of Pull Request

  • This is a bug fix.
  • This is an enhancement or feature.
  • This is a code style/formatting update.
  • This is a documentation update.
  • This is a refactoring update.
  • This is a chore update
  • This is something else.
    Please describe:

Related to Existing Issues

Test and Documentation Coverage

For bug fixes or features:

  • Tests have been completed.
  • Documentation has been added/updated.

Breaking Changes?

  • Yes, there are breaking changes.
  • No, there are no breaking changes.

@tenthirtyam tenthirtyam added the chore Chore label Oct 12, 2024
@tenthirtyam tenthirtyam requested a review from annakhm October 12, 2024 03:14
@tenthirtyam tenthirtyam self-assigned this Oct 12, 2024
@tenthirtyam tenthirtyam added needs-review Needs Review size/l Relative Sizing: large labels Oct 12, 2024
main.go Outdated Show resolved Hide resolved
@tenthirtyam tenthirtyam force-pushed the chore/update-copyright-and-spdx branch 3 times, most recently from cd67b5f to b13df72 Compare October 14, 2024 15:08
@tenthirtyam tenthirtyam requested a review from ksamoray October 14, 2024 15:09
tools/terragen.py Outdated Show resolved Hide resolved
@ksamoray
Copy link
Collaborator

@tenthirtyam note that we have a feature branch, vpc20-feature-branch, which adds a whole bunch of other files which should be modified as well.
IDK what's the best approach here - wait with such change until we merge vpc20-feature-branch into master, modify the addl files in the feature branch or cherry-pick this into the feature branch... Maybe @annakhm or @salv-orlando have a suggestion here.

@tenthirtyam tenthirtyam force-pushed the chore/update-copyright-and-spdx branch from b13df72 to 4e60c2a Compare October 14, 2024 15:21
@tenthirtyam
Copy link
Collaborator Author

tenthirtyam commented Oct 14, 2024

Hey @ksamora - 👋 I'd suggest waiting until the vpc20-feature-branch branch is merged and we can then rebase this branch and update any additional files that need the updates from vpc20-feature-branch to keep in all in one related change.

Ryan

@tenthirtyam tenthirtyam force-pushed the chore/update-copyright-and-spdx branch from 4e60c2a to af0a8f2 Compare January 8, 2025 14:45
@salv-orlando
Copy link
Member

@ksamoray It seems the linter error reported here is valid.
Probably not flagged by other PRs because they're not touching this file?

@salv-orlando
Copy link
Member

@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.
Perhaps it might be worth refreshing this PR to ensure the header is updated for all code modules?

@tenthirtyam
Copy link
Collaborator Author

Sure. I'll review and update these to align.

@tenthirtyam tenthirtyam force-pushed the chore/update-copyright-and-spdx branch from af0a8f2 to 29654e3 Compare January 8, 2025 15:58
@tenthirtyam
Copy link
Collaborator Author

@salv-orlando - updated the new files in 29654e3.

@ksamoray
Copy link
Collaborator

ksamoray commented Jan 8, 2025

@ksamoray It seems the linter error reported here is valid. Probably not flagged by other PRs because they're not touching this file?

Indeed, we can remove that func.
I'll submit a PR.

@tenthirtyam tenthirtyam force-pushed the chore/update-copyright-and-spdx branch from 29654e3 to ee77379 Compare January 15, 2025 14:30
@tenthirtyam tenthirtyam requested a review from ksamoray January 15, 2025 15:24
@tenthirtyam tenthirtyam added this to the v3.9.0 milestone Jan 15, 2025
@ksamoray
Copy link
Collaborator

Old copyright still appears in a bunch of files, e.g:

./tools/diffy/diffy.py:# Copyright (c) 2023 VMware, Inc. All Rights Reserved.
./nsxt/data_source_nsxt_vpc_service_profile.go:/* Copyright © 2020 VMware, Inc. All Rights Reserved.
./nsxt/data_source_nsxt_policy_gateway_connection.go:/* Copyright © 2022 VMware, Inc. All Rights Reserved.
./nsxt/data_source_nsxt_policy_distributed_vlan_connection.go:/* Copyright © 2022 VMware, Inc. All Rights Reserved.
./nsxt/data_source_nsxt_policy_transit_gateway.go:/* Copyright © 2020 VMware, Inc. All Rights Reserved.
./nsxt/data_source_nsxt_vpc_group.go:/* Copyright © 2020 VMware, Inc. All Rights Reserved.
./nsxt/data_source_nsxt_vpc_connectivity_profile.go:/* Copyright © 2020 VMware, Inc. All Rights Reserved.

@tenthirtyam
Copy link
Collaborator Author

I'll take another pass today.

@tenthirtyam tenthirtyam force-pushed the chore/update-copyright-and-spdx branch from ee77379 to 21e2764 Compare January 16, 2025 15:04
@tenthirtyam
Copy link
Collaborator Author

Old copyright still appears in a bunch of files, e.g:

./tools/diffy/diffy.py:# Copyright (c) 2023 VMware, Inc. All Rights Reserved. ./nsxt/data_source_nsxt_vpc_service_profile.go:/* Copyright © 2020 VMware, Inc. All Rights Reserved. ./nsxt/data_source_nsxt_policy_gateway_connection.go:/* Copyright © 2022 VMware, Inc. All Rights Reserved. ./nsxt/data_source_nsxt_policy_distributed_vlan_connection.go:/* Copyright © 2022 VMware, Inc. All Rights Reserved. ./nsxt/data_source_nsxt_policy_transit_gateway.go:/* Copyright © 2020 VMware, Inc. All Rights Reserved. ./nsxt/data_source_nsxt_vpc_group.go:/* Copyright © 2020 VMware, Inc. All Rights Reserved. ./nsxt/data_source_nsxt_vpc_connectivity_profile.go:/* Copyright © 2020 VMware, Inc. All Rights Reserved.

Resolved in 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]>
@tenthirtyam tenthirtyam force-pushed the chore/update-copyright-and-spdx branch from 21e2764 to e462f05 Compare January 20, 2025 21:35
@tenthirtyam tenthirtyam merged commit 4787a4b into master Jan 20, 2025
4 of 5 checks passed
@tenthirtyam tenthirtyam deleted the chore/update-copyright-and-spdx branch January 20, 2025 21:36
@tenthirtyam tenthirtyam removed the needs-review Needs Review label Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Chore size/l Relative Sizing: large
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants