Conversation
Post-transfer update of the copyright and SPDX headers. Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
There was a problem hiding this comment.
Pull request overview
Updates source file headers post-transfer by replacing legacy HashiCorp copyright notices with Broadcom copyright language while retaining the existing MPL-2.0 SPDX identifiers.
Changes:
- Replace
Copyright (c) HashiCorp, Inc.headers with Broadcom header text across Go, shell, HCL, YAML, and PowerShell example/test files. - Keep
SPDX-License-Identifier: MPL-2.0headers intact. - Minor whitespace cleanup in
.web-docs/scripts/compile-to-webdocs.shnear the header block.
Reviewed changes
Copilot reviewed 20 out of 20 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| testing/vsphere/simulator.go | Update file header to Broadcom + SPDX. |
| testing/vsphere/helpers.go | Update file header to Broadcom + SPDX. |
| examples/driver/main.go | Update file header to Broadcom + SPDX. |
| examples/builder/vsphere-supervisor/nginx-template.pkr.hcl | Update file header to Broadcom + SPDX. |
| examples/builder/vsphere-supervisor/cleanup-playbook.yml | Update file header to Broadcom + SPDX. |
| examples/builder/vsphere-iso/windows/windows-10.pkr.hcl | Update file header to Broadcom + SPDX. |
| examples/builder/vsphere-iso/windows/setup/setup.ps1 | Update file header to Broadcom + SPDX. |
| examples/builder/vsphere-iso/ubuntu/ubuntu-16.04.pkr.hcl | Update file header to Broadcom + SPDX. |
| examples/builder/vsphere-iso/alpine/setup.sh | Update file header to Broadcom + SPDX (after shebang). |
| examples/builder/vsphere-iso/alpine/alpine.pkr.hcl | Update file header to Broadcom + SPDX. |
| examples/builder/vsphere-clone/alpine/alpine.pkr.hcl | Update file header to Broadcom + SPDX. |
| datasource/virtualmachine/filters.go | Update file header to Broadcom + SPDX. |
| datasource/virtualmachine/data_test.go | Update file header to Broadcom + SPDX. |
| datasource/virtualmachine/data.go | Update file header to Broadcom + SPDX. |
| builder/vsphere/driver/storage_drs.go | Update file header to Broadcom + SPDX. |
| builder/vsphere/driver/datastore_cluster_mock.go | Update file header to Broadcom + SPDX. |
| builder/vsphere/driver/datastore_cluster.go | Update file header to Broadcom + SPDX. |
| builder/vsphere/common/step_resolve_datastore_test.go | Update file header to Broadcom + SPDX. |
| builder/vsphere/common/step_resolve_datastore.go | Update file header to Broadcom + SPDX. |
| .web-docs/scripts/compile-to-webdocs.sh | Update header to Broadcom + SPDX and remove an extra blank line. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // © Broadcom. All Rights Reserved. | ||
| // The term "Broadcom" refers to Broadcom Inc. and/or its subsidiaries. | ||
| // SPDX-License-Identifier: MPL-2.0 |
There was a problem hiding this comment.
PR description indicates a post-transfer update of copyright/SPDX headers, but the repo still contains at least one remaining HashiCorp header (e.g., .github/release.yml:1). Consider updating that file as part of this PR, or clarify in the PR summary that only a subset of headers are being updated.
|
This functionality has been released in v2.1.0 of the plugin. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
1 similar comment
|
This functionality has been released in v2.1.0 of the plugin. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
|
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Summary
Post-transfer update of the copyright and SPDX headers.
Type
fix: Bug Fixfeat: Feature or Enhancementdocs: Documentationrefactor: Refactoringchore: Build, Dependencies, Workflows, etc.other: Other (Please describe.)Breaking Changes?
Tests
Output:
Documentation
Issue References
Release Note
Additional Information