Skip to content
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

use go embed for templates #725

Merged
merged 1 commit into from Nov 15, 2021
Merged

use go embed for templates #725

merged 1 commit into from Nov 15, 2021

Conversation

bakito
Copy link
Contributor

@bakito bakito commented Nov 12, 2021

Since go 1.15 is now phased out we could reactivate the changes from #625

Use go embed feature to make template maintenance easier.

report/html/writer.go Show resolved Hide resolved
report/text/writer.go Show resolved Hide resolved
@ccojocar ccojocar merged commit ff17c30 into securego:master Nov 15, 2021
fnaranjo-vmw added a commit to cloudfoundry/persi-ci that referenced this pull request Nov 25, 2021
Latest changes in gosec don't work with go <1.16:
securego/gosec#725

We need more careful testing before doing such version bump
Cryogenics-CI pushed a commit to cloudfoundry/persi-ci that referenced this pull request Nov 25, 2021
Latest changes in gosec don't work with go <1.16:
securego/gosec#725

We need more careful testing before doing such version bump
fnaranjo-vmw added a commit to cloudfoundry/persi-ci that referenced this pull request Nov 25, 2021
Latest changes in gosec don't work with go <1.16:
securego/gosec#725

We need more careful testing before doing such version bump
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants