refactor Makefile - update comments

Author: prateekchaudhry

Makefile

@@ -162,18 +162,14 @@ endif
 test-ebs-csi:
 	make -C ./ecs-agent/daemonimages/csidriver test
 
-# Filter out packages that should not count toward coverage measurement.
-# Go's coverage redesign (default since Go 1.22+) instruments all packages in the module,
-# diluting coverage % by including mock/generated/untested packages in the denominator.
-# COVERPKG_EXCLUDE removes mock and generated packages.
-# The notest-packages.txt allowlists in scripts/coverfilters/ define packages permitted
-# to have no tests (interfaces, type definitions, generated code, main entrypoints, or
-# thin OS wrappers with no testable logic). New packages without tests will inflate the
-# denominator and fail CI, forcing developers to either:
-# 1. Add unit tests for the package (preferred), or
-# 2. Add it to the allowlist with a justification in the PR.
-# TODO: Add unit tests for allowlisted packages with real logic, such as
-# agent/eni/pause, ecs-agent/awsrulesfn, ecs-agent/metrics, ecs-agent/utils/httpproxy.
+# -coverpkg controls which packages Go instruments for coverage measurement.
+# It enables cross-package coverage: if package A's tests call package B,
+# that counts toward B's coverage. Without -coverpkg, packages with no tests
+# report 0% and dilute the overall coverage.
+# COVERPKG_EXCLUDE filters out mock and generated packages from the -coverpkg list.
+# The notest-packages.txt files in scripts/coverfilters/ list packages allowed to have
+# no tests. New untested packages will lower coverage and may fail CI — either
+# add tests (preferred) or add to the allowlist with justification in the PR.
 COVERPKG_EXCLUDE = grep -v -e '/mock' -e '/version/gen$$' | tr '\n' ','
 
 test: test-ebs-csi