search

chainguard-dev / bincapz

detect malicious program behaviors
Apache License 2.0
379 stars 24 forks source link

Makefile: add 'refresh-sample-testdata' rule #246

Closed tstromberg closed 1 month ago

tstromberg commented 1 month ago

Updating sample testdata can be confusing for new contributors. This should make it easier.

NOTE: Occasionally, this may generate an unexpected whitespace diff for markdown samples. Our markdown tests are whitespace insensitive.

tstromberg commented 1 month ago

One thought: should the test target call this target first so that the test data is always valid?

It sounds like a good idea, but in my experience, any time something named "test" modifies data, developers find it to be an unexpected and baffling behavior.

egibs commented 1 month ago

It sounds like a good idea, but in my experience, any time something named "test" modifies data, developers find it to be an unexpected and baffling behavior.

True indeed. This is always one make command away so that's still convenient enough. 🚀