-
Notifications
You must be signed in to change notification settings - Fork 304
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Default loading path for data
?
#800
Comments
This seems reasonable to me. It would be straightforward to set default values for the |
Hey, I want to pick up this issue. |
Add default value as 'data' for the data flag. Updated data loading function to not throw an error if it is using the default data path and it is missing. Fixes open-policy-agent#800
I saw one or two very old comments about having a default
data
path (likepolicy/data
or justdata
), but it seems like there isn't one yet.#215 (review)
#169 (comment)
For me, this makes
bundles
hard to use effectively, because the bundle isn't enough, you need to make sure that the consumingconftest
call adds a-d
flag. If there was a defaultpolicy/data
folder, then I could publish bundles and consume them easily. (Note: the --combine flag is one obstacle, but can deal with that problem later)The text was updated successfully, but these errors were encountered: