Changes are broken out into two commits
The first adds config files, updates ci.yml, adds the git attributes, and updates the README.
The second updates the repo to pass the linting stage.
Add CI step to verify go generate was run on repo.
Add linter stage to CI along with linter config file, .golangci.yml.
Updated README.md Contributing section on linting requirements.
Added .gitattributes file to prevent issues with checkout out CRLF and gofmt
Added sequence ordering to make sure lint and go generate stages run before tests and build.
This way, build and tests are not run on code that could potentially:
not build due to gofmt issues;
contain bugs;
have to be re-submitted after issues are fixed; or
contain outdated Win32 syscall or other auto-generated files.
Changes are broken out into two commits The first adds config files, updates
ci.yml, adds the git attributes, and updates the README. The second updates the repo to pass the linting stage.Add CI step to verify
go generatewas run on repo. Add linter stage to CI along with linter config file,.golangci.yml.Updated
README.mdContributing section on linting requirements.Added
.gitattributesfile to prevent issues with checkout outCRLFandgofmtAdded sequence ordering to make sure lint and go generate stages run before tests and build. This way, build and tests are not run on code that could potentially:
gofmtissues;