refactor: move checkcover to internal/cmd and gitignore binary

Addresses PR feedback from @dmathieu:
- Move checkcover from cmd/ to internal/cmd/ (not needed as public tool)
- Remove committed binary from git tracking
- Add binary to .gitignore to prevent future commits

Changes:
- Moved cmd/checkcover -> internal/cmd/checkcover
- Updated module path in go.mod
- Updated CI workflow to reference new path
- Updated documentation references
- Added checkcover binaries to .gitignore
- All tests passing

Author: sAchin-680

.github/workflows/build-and-test.yml

@@ -227,8 +227,8 @@ jobs:
           CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
       - name: Validate coverage requirements
         run: |
-          cd cmd/checkcover
-          go run . -c ../../coverage.txt -r ../.. -v
+          cd internal/cmd/checkcover
+          go run . -c ../../../coverage.txt -r ../../.. -v
 
   cross-build-collector:
     needs: [setup-environment]

.gitignore

@@ -2,6 +2,10 @@ bin/
 dist/
 /local
 
+# Compiled binaries
+cmd/checkcover/checkcover
+internal/cmd/checkcover/checkcover
+
 # GoLand IDEA
 /.idea/
 *.iml

PR_SUMMARY.md

@@ -32,12 +32,12 @@ Created a new CLI tool that:
 
 Files:
 
-- `cmd/checkcover/main.go` - CLI entry point
-- `cmd/checkcover/validator.go` - Core logic
-- `cmd/checkcover/validator_test.go` - Tests
-- `cmd/checkcover/README.md` - Documentation
-- `cmd/checkcover/go.mod` - Dependencies
-- `cmd/checkcover/Makefile` - Build config
+- `internal/cmd/checkcover/main.go` - CLI entry point
+- `internal/cmd/checkcover/validator.go` - Core logic
+- `internal/cmd/checkcover/validator_test.go` - Tests
+- `internal/cmd/checkcover/README.md` - Documentation
+- `internal/cmd/checkcover/go.mod` - Dependencies
+- `internal/cmd/checkcover/Makefile` - Build config
 
 ### 4. CI Integration
 
@@ -83,8 +83,8 @@ status:
 ```bash
 # Locally
 make gotest-with-cover
-cd cmd/checkcover
-go run . -c ../../coverage.txt -r ../.. -v
+cd internal/cmd/checkcover
+go run . -c ../../../coverage.txt -r ../../.. -v
 
 # In CI (automatic)
 # Runs as part of test-coverage job
@@ -163,7 +163,7 @@ None. This is purely additive:
 
 ## Notes for Reviewers
 
-1. **checkcover tool**: Main implementation in `cmd/checkcover/`
+1. **checkcover tool**: Main implementation in `internal/cmd/checkcover/`
 2. **Metadata changes**: Simple addition of one field with validation
 3. **CI changes**: Single step added to existing workflow
 4. **Documentation**: Comprehensive guide for component authors

cmd/checkcover/checkcover

@@ -102,8 +102,8 @@ You can check your component's coverage requirements and actual coverage:
 make gotest-with-cover
 
 # Run the coverage validator
-cd cmd/checkcover
-go run . -c ../../coverage.txt -r ../.. -v
+cd internal/cmd/checkcover
+go run . -c ../../../coverage.txt -r ../../.. -v
 ```
 
 ### In CI
@@ -159,7 +159,7 @@ status:
 If you encounter issues with coverage validation:
 
 1. Check the [Component Stability](../../docs/component-stability.md) document
-2. Review the [checkcover README](../../cmd/checkcover/README.md)
+2. Review the [checkcover README](../../internal/cmd/checkcover/README.md)
 3. Ask in the #otel-collector Slack channel
 4. Open an issue on GitHub with the label `coverage`
 

docs/coverage-requirements.md

@@ -1,4 +1,4 @@
-module go.opentelemetry.io/collector/cmd/checkcover
+module go.opentelemetry.io/collector/internal/cmd/checkcover
 
 go 1.24.0
 
@@ -27,8 +27,10 @@ require (
 	go.uber.org/zap v1.27.0 // indirect
 )
 
-replace go.opentelemetry.io/collector/component => ../../component
+replace go.opentelemetry.io/collector/component => ../../../component
 
-replace go.opentelemetry.io/collector/config/configtelemetry => ../../config/configtelemetry
+replace go.opentelemetry.io/collector/config/configtelemetry => ../../../config/configtelemetry
 
-replace go.opentelemetry.io/collector/pdata => ../../pdata
+replace go.opentelemetry.io/collector/pdata => ../../../pdata
+
+replace go.opentelemetry.io/collector/featuregate => ../../../featuregate