Skip to content

wip make policies not apply to the wrapper

Rodrigo Muino Tomonari requested to merge github/fork/bmeck/policy-fix-eval-wrapper into master

This PR makes it so policies don't have to apply to the wrapper for things like STDIN or --eval.

Currently this does keep the URL for the check at file:///...[eval] etc. Likely we should differentiate this on the policy rather than having it collide with a file on disk. I was thinking something like app:{[eval],[stdin]}. However, this does have a problem that the wrapper module path would differ from the policy. I'm open to any sort of disambiguation or punting that for now even.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

Merge request reports