-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add workflows to improve the ease of open source contributions. #158
Comments
When it comes to unit tests we have the github actions that run the unit tests for linux and windows on every PR - is there anything else we'd need there? |
I think unit tests are covered, I missed those when initially writing this. Though I think we need to add macOS as a platform to test as well? |
Also need to add
|
Hi @another-rex @erikvarga can i pick this one up ? |
Sure, my original idea is to configure golangci-lint to perform this lint, but turns out to be harder to enable than expected: golangci/golangci-lint#456 |
So , then what's your uptake on this ? |
Non-exhaustive list of workflows to add:
The text was updated successfully, but these errors were encountered: