Skip to content
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

Caveats support #88

Open
michelkaeser opened this issue Nov 15, 2023 · 4 comments
Open

Caveats support #88

michelkaeser opened this issue Nov 15, 2023 · 4 comments

Comments

@michelkaeser
Copy link

See https://authzed.com/docs/reference/caveats for the official documentation.

The attached patch might be helpful as it contains the ground work for this feature.

caveats.patch

Sadly I have issues to build locally so I would be grateful if someone else could implement this.
Many thanks.

@iocanel
Copy link
Contributor

iocanel commented Nov 27, 2023

What kind of issues do you have?

@michelkaeser
Copy link
Author

michelkaeser commented Nov 27, 2023

warning: the following paths have collided (e.g. case-sensitive paths
on a case-insensitive filesystem) and only one from the same
colliding group is in the working tree:

  'grpc/client/src/main/java/com/authzed/api/v1alpha1/WatchResourcesService.java'
  'grpc/client/src/main/java/com/authzed/api/v1alpha1/WatchresourcesService.java'

which, with bin/generate.sh later leads to

[ERROR] /quarkus-authzed-client/grpc/client/src/main/java/com/authzed/api/v1alpha1/WatchResourcesService.java:[6,14] class WatchresourcesService is public, should be declared in a file named WatchresourcesService.java

Tried to work around it but couldn't.

@iocanel
Copy link
Contributor

iocanel commented Nov 27, 2023

Let me try that

1 similar comment
@iocanel
Copy link
Contributor

iocanel commented Nov 27, 2023

Let me try that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants