search

Melkeydev / go-blueprint

Go-blueprint allows users to spin up a quick Go project using a popular framework
MIT License
2.07k stars 141 forks source link

Fix: Add build step for generated project #196

Closed MitchellBerend closed 1 month ago

MitchellBerend commented 2 months ago

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

Problem/Feature

Resolves #195

In #174 I had some syntax errors in the templates that ended up in the final generated code. Since all the CI tests passed I assumed all the projects were at least compiling. Turns out this assumption was wrong.

Description of Changes:

This pr adds the advanced flag to the linting matrix step in CI that checks the generated project. This checks for typos of just general mistakes in syntax in the generated code of the advanced features as well.

This pr also fixes all the linting errors that popped up by adding the advanced features to the checks.

Checklist

MitchellBerend commented 2 months ago

I also took the liberty of bumping the setup-go, golangci-lint and cache steps to get rid of some of the warnings about node 16 being deprecated.