This matches the linting/formatting setup to the one used in CLI, namely:
Add .golangci.yaml file that matches the ruleset there
Removes gci usage as it's integrated into golangci-lint
It also prevents any formatting of the generated directory to keep things simple, if we add a validator package we just need to update the gofumpt command.
It does not deal with any fixes yet, there's a good amount that's auto-fixable but around 80 or so to manually fix (a range of whitespace, line length, naming, unused args, and general code), so waiting for a 👍🏻 or 👎🏻 consensus before tackling those
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[x] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected
Description
This matches the linting/formatting setup to the one used in CLI, namely:
.golangci.yaml
file that matches the ruleset theregci
usage as it's integrated into golangci-lintIt also prevents any formatting of the generated directory to keep things simple, if we add a
validator
package we just need to update thegofumpt
command.It does not deal with any fixes yet, there's a good amount that's auto-fixable but around 80 or so to manually fix (a range of whitespace, line length, naming, unused args, and general code), so waiting for a 👍🏻 or 👎🏻 consensus before tackling those
References
Review Checklist
main