Skip to content

Enable lint failures in analysis workflow#3094

Open
Codex wants to merge 8 commits intomainfrom
codex/remove-linting-error-suppression
Open

Enable lint failures in analysis workflow#3094
Codex wants to merge 8 commits intomainfrom
codex/remove-linting-error-suppression

Conversation

@Codex
Copy link
Copy Markdown
Contributor

@Codex Codex AI commented Mar 6, 2026

Analysis workflow was suppressing lint errors, allowing super-linter findings to pass silently.

  • Workflow: Set DISABLE_ERRORS: 'false' in .github/workflows/analyze.yml to allow lint failures to fail the job.

Example:

- uses: super-linter/super-linter@v8.5.0
  env:
    DISABLE_ERRORS: 'false'
Original prompt

This section details on the original issue you should resolve

<issue_title>Remove the suppression of linting errors in the analysis workflow</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @codex[agent] in this section)

@Codex Codex AI changed the title [WIP] Remove the suppression of linting errors in analysis workflow Enable lint failures in analysis workflow Mar 6, 2026
@plengauer plengauer marked this pull request as ready for review March 6, 2026 16:33
@plengauer plengauer requested a review from moflwi as a code owner March 6, 2026 16:33
Copilot AI review requested due to automatic review settings March 6, 2026 16:33
@plengauer plengauer enabled auto-merge (squash) March 6, 2026 16:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes suppression of super-linter failures in the analysis workflow so lint findings correctly fail the CI job.

Changes:

  • Set DISABLE_ERRORS to 'false' in the analysis workflow to stop ignoring linter errors.

Comment thread .github/workflows/analyze.yml
moflwi
moflwi previously approved these changes Mar 9, 2026
@plengauer plengauer enabled auto-merge (squash) March 20, 2026 22:31
@plengauer plengauer disabled auto-merge March 20, 2026 22:31
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.

Remove the suppression of linting errors in the analysis workflow

4 participants