Skip to content

fix: make lint/go for non-interactive environments#6448

Merged
Warashi merged 3 commits into
pipe-cd:masterfrom
Parship12:fix-make-lint
Mar 9, 2026
Merged

fix: make lint/go for non-interactive environments#6448
Warashi merged 3 commits into
pipe-cd:masterfrom
Parship12:fix-make-lint

Conversation

@Parship12
Copy link
Copy Markdown

@Parship12 Parship12 commented Jan 21, 2026

What this PR does:
Fix make lint/go for non-interactive environments.

Why we need it:
make lint/go was failing in non-interactive environments (like ci or automated scripts) due to the -it flag in the docker command, which causing "the input device is not a TTY" error. The lint output is shown in the console, but the command returns a failure exit code.

Changes:

  • Removed -it flag from lint/go.

Testing:

  • make lint/go: complete with exit code 0
  • 13 Go modules report 0 linting issues

Before:
Screenshot 2026-01-25 161828

After:
Screenshot 2026-01-25 161917

Note: CI workflow already has working golangci-lint integration with reviewdog and uses a matrix strategy for parallel execution, which is optimal for CI. This PR focuses on fixing local/scripted usage of make lint/go.

Does this PR introduce a user-facing change?: No

  • How are users affected by this change: This change only affects developers running make lint/go locally or in scripts.
  • Is this breaking change: No
  • How to migrate (if breaking change): NA

Signed-off-by: Parship Chowdhury <parshipchowdhury@gmail.com>
@Parship12 Parship12 requested a review from a team as a code owner January 21, 2026 18:03
@eeshaanSA
Copy link
Copy Markdown
Contributor

cc: @khanhtc1202 @Warashi

Copy link
Copy Markdown
Contributor

@Ayushmore1214 Ayushmore1214 left a comment

Choose a reason for hiding this comment

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

Hey @Parship12, thanks for this! However, I'd suggest we revert the changes to lint.yaml the current setup uses a matrix to run linting in parallel (which is faster) and uses reviewdog to post comments directly on the code. If we switch to a single make lint/go command in CI, we lose that parallelism and the inline feedback, which makes debugging harder for contributors, So Let's keep the Makefile fix and kep the Github workflow as is , otherwise nice!

@Parship12
Copy link
Copy Markdown
Author

Good point. I will revert the lint.yaml changes. Thanks

Signed-off-by: Parship Chowdhury <parshipchowdhury@gmail.com>
@Parship12 Parship12 changed the title fix: make lint/go for non-interactive environments and simplify ci fix: make lint/go for non-interactive environments Jan 25, 2026
@Parship12
Copy link
Copy Markdown
Author

Parship12 commented Jan 25, 2026

While testing issue #4566, I discovered the -it flag issue (fixed in this PR). I believe #4566/#4566 (comment) can be closed as I think both the todos are done (make lint/go working with no errors and CI already has a better solution).

@Parship12
Copy link
Copy Markdown
Author

@eeshaanSA @khanhtc1202 PTAL

@Ayushmore1214
Copy link
Copy Markdown
Contributor

LGTM

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 3, 2026

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale label Mar 3, 2026
Copy link
Copy Markdown
Member

@Warashi Warashi left a comment

Choose a reason for hiding this comment

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

Sorry for late review, LGTM

@Warashi Warashi enabled auto-merge (squash) March 9, 2026 20:59
@Warashi Warashi merged commit bededb6 into pipe-cd:master Mar 9, 2026
45 checks passed
@Parship12 Parship12 deleted the fix-make-lint branch March 10, 2026 01:58
@github-actions github-actions Bot mentioned this pull request May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants