Skip to content

ci(file-manager): enable lint_fail_on_issues#424

Open
BjoernKarma wants to merge 1 commit into
mainfrom
feat/lint-file-manager
Open

ci(file-manager): enable lint_fail_on_issues#424
BjoernKarma wants to merge 1 commit into
mainfrom
feat/lint-file-manager

Conversation

@BjoernKarma

Copy link
Copy Markdown
Contributor

Enable lint_fail_on_issues for the file-manager domain. The module already passes golangci-lint with 0 issues

Copilot AI review requested due to automatic review settings June 11, 2026 09:52
@BjoernKarma BjoernKarma added lint Anything related to the linter file-manager labels Jun 11, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR tightens CI enforcement for the file-manager module by making the workflow fail when golangci-lint reports any issues, aligning it with several other modules that already enforce this behavior.

Changes:

  • Enable lint_fail_on_issues: true for the file-manager job in the CI workflow.
  • Ensure file-manager CI lint behavior matches other domains already configured to fail on lint issues.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Enable lint_fail_on_issues for the file-manager domain.
The module already passes golangci-lint with 0 issues
@BjoernKarma BjoernKarma force-pushed the feat/lint-file-manager branch from d4bf19a to d960786 Compare June 11, 2026 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

file-manager lint Anything related to the linter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants