10 Commits
Author SHA1 Message Date
Julio López fdf0159a17 refactor(general): cleanup various nits (#5618)
- use struct field initialization idiom
- clarify error message
- directly use s.rootctx in server request processing
- refactor: use require in e2e ACL test
- inline constant definition
- modernize with `strings.Cut`
- modernize with slices.Backward
- simplify timeFormat initialization
- refactor testing: check `Scanner.Err()` in testenv
- remove spurious parenthesis
- trueStr and falseStr consts
- use inheritPolicyString
- use const in cache info
- "bytes" const in ui task counters
- use common consts in gettool
2026-09-06 15:08:49 -07:00
Julio López 78407b1d5f refactor(general): string nits (#5265)
Minor improvements:

Refactor the construction of a test string with many control characters in `TestJSONWriter_StringEscapingPerformanceWithManyControlChars` by using a `strings.Builder`.
Rationale: address a future `modernize` linter warning. While this is also "more" efficient, it does not make any practical difference in this context.

Simplify the conversion of a short flag to a string in `emitFlags` by removing an unnecessary byte slice conversion (`tools/cli2md/cli2md.go`).
Rationale: simplify the expression for clarity and address a future linter warning.

- Ref: #5247
2026-04-01 20:50:10 -07:00
Julio Lopez 9c1aec59e4 docs(site): fix CLI command docs (#4979)
Also, minor updates for contrib docs
2025-11-12 17:19:34 -08:00
Nathan Baulch 657fda216a chore(ci): upgrade to golangci-lint 2.6.1 (#4973)
- upgrade to golangci-lint 2.6.1
- updates for gosec
- updates for govet
- updates for perfsprint
- updates modernize

Leaves out modernize:omitempty due to conflicts with tests
2025-11-11 21:27:10 -08:00
blenderfreakyandJulio Lopez 6cd728394d fix(site): escape flags with backticks (#4479)
* fix(site): escape flags with backticks

In the generated markdown docs, flags like `--foo` inside
help texts currently get pretty-printed as `–foo` with an em-dash.

This change applies backticks via a regex replacement, so that
they appear as `--foo` in the docs but remain as --foo in the
CLI output.

---------

Co-authored-by: Julio Lopez <1953782+julio-lopez@users.noreply.github.com>
2025-04-23 23:42:29 -07:00
Matthieu MOREL 675e958877 chore(ci): bump golangci-lint to v2.1.2 (#4500)
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
2025-04-15 22:49:13 -07:00
Jarek Kowalski fcb8197f3f chore(ci): upgraded linter to 1.59.0 (#3883) 2024-05-29 20:31:57 -07:00
Christoph AndersonandlupusA 7b89fe173c Removed summary in command section (#3590)
Co-authored-by: lupusA <lupuapps@gmail.com>
2024-01-26 11:06:24 -08:00
Jarek Kowalski cbc66f936d chore(ci): upgraded linter to 1.53.3 (#3079)
* chore(ci): upgraded linter to 1.53.3

This flagged a bunch of unused parameters, so the PR is larger than
usual, but 99% mechanical.

* separate lint CI task

* run Lint in separate CI
2023-06-18 13:26:01 -07:00
Jarek Kowalski 5cf54906ec chore(site): upgraded site to latest hugo and removed vendored files (#3078)
* chore(site): upgraded site to latest hugo and removed vendored files

This switches to using github.com/google/docsy as a Hugo module,
so we don't need to vendor the theme anymore or its dependencies.

* layout

* more style, edit MD support

* style

* style

* fixed linter errors

* update text
2023-06-18 11:41:01 -07:00