villebro opened a new pull request, #332:
URL: https://github.com/apache/superset-kubernetes-operator/pull/332

   ## Summary
   
   The Fuzzing section of the development guidelines told contributors, on a 
fuzz finding, to *commit the raw `testdata/fuzz/<target>/<id>` reproducer* as a 
permanent regression seed. That doesn't work in this repo: Go's generated 
corpus files carry no Apache license header and fail the `check-license` 
(Apache Rat) job, and Rat can't reliably exclude them because it matches on 
their hash-based filenames.
   
   This updates the guidance to the approach that actually holds: capture the 
reproducer's minimized input and add it back as an `f.Add(...)` seed in the 
target (a license-headed Go source line that replays on every `make 
test-unit`), then delete the generated `testdata/fuzz` file and fix the bug. 
Same regression coverage, no license-check friction, and the seed is 
self-documenting next to a comment.
   
   ## Details
   
   - Rewrites the "On a finding" instruction in 
`docs/contributing/development-guidelines.md`.
   - Drops the "(plus any committed `testdata/fuzz/...` reproducers)" clause 
from the corpus-replay sentence, since we no longer commit those files.
   
   No code or behavior change.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to