Skip to content

Harden Super Linter workflow against generated-summary linting and log artifact permission failures#41729

Merged
pelikhan merged 2 commits into
mainfrom
copilot/fix-superlinter-docs
Jun 26, 2026
Merged

Harden Super Linter workflow against generated-summary linting and log artifact permission failures#41729
pelikhan merged 2 commits into
mainfrom
copilot/fix-superlinter-docs

Conversation

Copilot AI commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

The scheduled Super Linter Report workflow was failing for two independent reasons: super-linter attempted to process its own generated summary output, and the post-step log artifact upload failed on file ownership/permission boundaries.

  • Scope of fix

    • Exclude generated super-linter output from markdown validation with FILTER_REGEX_EXCLUDE.
    • Remove step-summary generation env wiring from the super-linter step to avoid the summary formatting failure path in this workflow mode.
  • Log artifact reliability

    • Replace direct chmod/upload of super-linter.log with a staged copy to /tmp/super-linter.log.
    • Emit explicit warnings on copy/chmod failure (instead of silent suppression).
    • Make artifact upload non-fatal when the staged file is missing (if-no-files-found: ignore).
  • Compiled workflow parity

    • Regenerate .github/workflows/super-linter.lock.yml from the markdown workflow so runtime behavior matches source.
env:
  VALIDATE_MARKDOWN: "true"
  VALIDATE_ALL_CODEBASE: "false"
  FILTER_REGEX_EXCLUDE: "(^|.*/)super-linter-output/.*"

Copilot AI and others added 2 commits June 26, 2026 15:03
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Fix Super Linter markdown summary and log artifact failures Harden Super Linter workflow against generated-summary linting and log artifact permission failures Jun 26, 2026
Copilot AI requested a review from pelikhan June 26, 2026 15:10
@pelikhan pelikhan marked this pull request as ready for review June 26, 2026 15:51
Copilot AI review requested due to automatic review settings June 26, 2026 15:51

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 hardens the scheduled Super Linter Report workflow so it no longer self-lints generated output and so log artifact upload is more reliable across file ownership/permission boundaries. It updates both the markdown source workflow and the compiled lock workflow to keep runtime behavior aligned with the authored source.

Changes:

  • Exclude generated super-linter-output/ artifacts from Markdown validation via FILTER_REGEX_EXCLUDE.
  • Disable Super Linter’s step-summary env wiring in this workflow mode to avoid the summary formatting failure path.
  • Improve log upload robustness by staging the log to /tmp, emitting explicit warnings on failures, and making artifact upload non-fatal when missing.
Show a summary per file
File Description
.github/workflows/super-linter.md Adds exclusion regex for generated output and stages the log to /tmp for permission-safe artifact upload.
.github/workflows/super-linter.lock.yml Regenerates the compiled workflow to reflect the source workflow’s new env and log-upload behavior.

Review details

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Low

@pelikhan pelikhan merged commit 75354bc into main Jun 26, 2026
1 check passed
@pelikhan pelikhan deleted the copilot/fix-superlinter-docs branch June 26, 2026 15:55
@github-actions

Copy link
Copy Markdown
Contributor

@copilot please post a short completion plan or the remaining blockers for the Super Linter workflow changes.

Generated by 👨‍🍳 PR Sous Chef · 47.1 AIC · ⌖ 0.968 AIC · ⊞ 17.1K ·

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.

3 participants