Skip to content

chore: enable perfsprint and make changes#1045

Merged
catsby merged 3 commits intomainfrom
perfsprint
Feb 12, 2025
Merged

chore: enable perfsprint and make changes#1045
catsby merged 3 commits intomainfrom
perfsprint

Conversation

@catsby
Copy link
Copy Markdown
Contributor

@catsby catsby commented Dec 18, 2024

Description

Enable perfsprint linter and update usage of fmt.Sprintf accordingly

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@catsby catsby requested a review from a team as a code owner December 18, 2024 20:19
Signed-off-by: catsby <clint@defenseunicorns.com>
catsby added 2 commits February 12, 2025 10:55
* main:
  chore: set helm buildflags the way Zarf does (#1052)
  chore(deps): update github-actions (#1081)
  chore(deps): update github-actions (#1077)
  chore: copy over our own save-logs (#1075)
  chore(deps): update github-actions (#1072)
  chore: regroup and unlimit renovate (#1071)
  fix(deps): update module github.com/defenseunicorns/maru-runner to v0.5.0 (#1067)
  chore(config): migrate renovate config (#1069)
  chore: update zarf version ref to v0.48.1 (#1066)
  fix(deps): update golang.org/x/exp digest to 7d7fa50 (#1016)
  chore(deps): update actions/setup-go action to v5.2.0 (#1050)
  fix(deps): update module github.com/goccy/go-yaml to v1.15.12 (#1049)
* main:
  chore: go mod updates again (#1087)
Copy link
Copy Markdown
Member

@YrrepNoj YrrepNoj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with a quick learning question

Comment thread src/cmd/vendored.go
@catsby catsby merged commit b6e5e65 into main Feb 12, 2025
@catsby catsby deleted the perfsprint branch February 12, 2025 22:39
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.

2 participants