permitio / opal

Policy and data administration, distribution, and real-time updates on top of Policy Agents (OPA, Cedar, ...)
https://opal.ac
Apache License 2.0
5.14k stars 180 forks source link

fix failing pre-commit workflow #683

Closed daveads closed 1 week ago

daveads commented 1 month ago

Fixed pre-commit failling test...

netlify[bot] commented 1 month ago

Deploy Preview for opal-docs canceled.

Name Link
Latest commit 5149b230d12480d1da03fb12eee32b4c18a32a29
Latest deploy log https://app.netlify.com/sites/opal-docs/deploys/6738deafb8f21100086a7a68
daveads commented 1 month ago

docformatter works.. properly now.. @gemanor @danyi1212

This pr should be attended to be merging any other pr...

daveads commented 1 month ago

I got the insight to fix this from the following sources:

https://github.com/PyCQA/docformatter/pull/287
https://github.com/PyGithub/PyGithub/issues/3058
https://github.com/Lightning-AI/torchmetrics/pull/2779
https://github.com/PyCQA/docformatter/issues/289
daveads commented 1 month ago

@danyi1212 This is ready for merge...

daveads commented 1 week ago

@danyi1212 i think you should just merge this... probably Tests / test-docker (pull_request) only works on the master branch only.