DrString icon indicating copy to clipboard operation
DrString copied to clipboard

Published 20 hours ago • verified publisher icon dduan

ACL Filtering

Open rstone4-chwy opened this issue 3 years ago • 2 comments

My team is starting to use DrString, but we're hitting a scenario that the library doesn't seem to support. We care that our publicly documented methods and variables are correct, but methods with internal ACL do not need to be validated. Is it possible to only validate public variables and methods? If not would you accept a PR to implement such a behavior?

rstone4-chwy avatar Jul 18 '22 12:07 rstone4-chwy

A PR sounds great! Can you describe what you have in mind? Would it be

  • a binary flag that makes check/formatting/extracting public only (--public-only)
  • an option that specify the minimal level (--min-acl)
  • a list of ACLs (--acl public --acl internal)

No matter what we pick, the default behavior should remain the same.

dduan avatar Jul 18 '22 23:07 dduan

I hadn't gotten that far with the design. Passing a list of ACLS seems like it would be the most flexible. I don't imagine many people wanting to verify private docs but not public, but there's always someone who has a specific need.

rstone4-chwy avatar Jul 19 '22 13:07 rstone4-chwy