tektoncd / tektoncd/cli

Remove inline disable gosec lint error

Open
#2,740 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

kind/cleanup
Dominant language
Go
Stars
461
Forks
277
Avg merge
16h 48m
Merged PRs (30d)
95

Description

Remove all inline gosec disables from the code and perform the required refactoring to address the issues properly. (eg. gosec lint)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the gosec lint example in cmd/tkn/main.go at line 42, then search the repository for all inline gosec disables. Run the gosec lint checks to identify each underlying issue and inspect the affected code before refactoring it. Done means no inline gosec disables remain and the lint checks pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
cli, security
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.