generated from TrueCloudLab/basic
FrostFS Access Policy Engine
Evgenii Stratonikov
a03c00c44f
Some checks failed
DCO action / DCO (pull_request) Failing after 53s
Tests and linters / Tests (1.21) (pull_request) Successful in 48s
Tests and linters / Tests (1.20) (pull_request) Successful in 50s
Tests and linters / Tests with -race (pull_request) Successful in 1m11s
Tests and linters / Lint (pull_request) Successful in 1m41s
Tests and linters / Staticcheck (pull_request) Successful in 1m48s
So, it's sunday evening and I am sitting on-call trying to debug strange node behaviour. It took me 3 whole minutes to understand the code being changed: it accumulates bools in slices, even though no slice is needed; it uses subtle condition from the first loop to make decision in the second one, and finally it uses named return values. In this commit we remove the slice and the second loop, because why not. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com> |
||
|---|---|---|
| .forgejo/workflows | ||
| docs | ||
| iam | ||
| pkg | ||
| schema | ||
| util | ||
| .gitattributes | ||
| .gitignore | ||
| .gitlint | ||
| .golangci.yml | ||
| .pre-commit-config.yaml | ||
| go.mod | ||
| go.sum | ||
| Makefile | ||