Skip to content

[IND-4227][COMPLIANCE] chore: Fix lint issues-build tags#568

Merged
ssagarverma merged 1 commit intomainfrom
ssagarverma/fix-lint-issues
Nov 17, 2025
Merged

[IND-4227][COMPLIANCE] chore: Fix lint issues-build tags#568
ssagarverma merged 1 commit intomainfrom
ssagarverma/fix-lint-issues

Conversation

@ssagarverma
Copy link
Contributor

@ssagarverma ssagarverma commented Nov 10, 2025

This change is required to fix lint issues that appeared during CI checks in #566.

Replaced all deprecated // +build directives with the modern //go:build form
introduced in Go 1.17. This modernizes build constraints to align with the Go 1.24
toolchain and removes obsolete syntax flagged by go vet linter.

No runtime or functional behavior changes.

⚠️ Note: This change drops compatibility with Go versions earlier than 1.17,
as they do not support the //go:build syntax.

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

Update build tags to use the new syntax (go version>=1.17) and remove redundant tags.
@ssagarverma ssagarverma requested a review from a team as a code owner November 10, 2025 11:45
@hashicorp-cla-app
Copy link

hashicorp-cla-app bot commented Nov 10, 2025

CLA assistant check
All committers have signed the CLA.

@hashicorp-cla-app
Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

@ssagarverma ssagarverma changed the title [IND-4227][COMPLIANCE] Fix lint issues-build tags [IND-4227][COMPLIANCE] chore: Fix lint issues-build tags Nov 10, 2025
@ssagarverma ssagarverma merged commit cfc9f1a into main Nov 17, 2025
12 checks passed
@ssagarverma ssagarverma deleted the ssagarverma/fix-lint-issues branch November 17, 2025 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants