Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 7, 2025

Super Linter's LOG_FILE parameter expects a filename only, not a full path. The workflow was incorrectly configured with /tmp/gh-aw/super-linter.log, causing the linter to fail. Additionally, the super_linter job included unnecessary Go/Node.js setup steps that only the main agent job requires.

Changes

  • LOG_FILE configuration: Changed from /tmp/gh-aw/super-linter.log to super-linter.log (filename only)
  • Removed setup steps from super_linter job:
    • Set up Go
    • Set up Node.js
    • Install Dependencies (go mod download, npm ci)
    • Create log directory
  • Updated path references: All log file references in check-results and upload-artifact steps now use super-linter.log

The super_linter job now only checks out code and runs Super Linter, making it faster and correctly configured.

Original prompt

Fox the super linter configuration with the log file. It has to be a file name, not a path.

https://github.com/githubnext/gh-aw/actions/runs/19182860297

Also the setup install steps are not needed in the superlinter job


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…steps

- Changed LOG_FILE from path to filename only (super-linter.log instead of /tmp/gh-aw/super-linter.log)
- Removed unnecessary "Set up Go" step from super_linter job
- Removed unnecessary "Set up Node.js" step from super_linter job
- Removed unnecessary "Install Dependencies" step from super_linter job
- Removed unnecessary "Create log directory" step from super_linter job
- Updated all references to the log file path in check and upload steps

This fixes the issue where Super Linter expects LOG_FILE to be a filename only, not a full path.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review November 7, 2025 22:41
Copilot AI review requested due to automatic review settings November 7, 2025 22:41
Copy link
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

This PR simplifies the super-linter workflow by removing unnecessary build dependencies and changing the log file path from /tmp/gh-aw/super-linter.log to super-linter.log. The workflow previously set up Go and Node.js build environments that weren't needed since super-linter already includes all necessary linters.

Key Changes:

  • Removed Go and Node.js setup steps that were not required for linting operations
  • Simplified log file path from /tmp/gh-aw/super-linter.log to super-linter.log
  • Eliminated the unnecessary step to create the /tmp/gh-aw directory

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/super-linter-report.md Removed unnecessary Go/Node.js setup steps and simplified log file path to use working directory
.github/workflows/super-linter-report.lock.yml Compiled version of workflow with matching changes to dependencies and log paths
Comments suppressed due to low confidence (2)

.github/workflows/super-linter-report.md:101

  • The instructions reference the old log file path /tmp/gh-aw/super-linter.log, but the log file is now written to super-linter.log in the working directory. This mismatch will cause the AI agent to fail when trying to read the log file.
1. **Read the linter output** from `/tmp/gh-aw/super-linter.log` using the bash tool

.github/workflows/super-linter-report.md:80

  • The artifact download path still references /tmp/gh-aw/, but the uploaded artifact now contains super-linter.log from the working directory. This creates an inconsistency where the file is downloaded to /tmp/gh-aw/super-linter.log but the instructions in line 101 and the actual upload are based on different paths. The download path should be updated to match the new simplified approach or the instructions should reference the download location.
  - name: Download super-linter log
    uses: actions/download-artifact@v4
    with:
      name: super-linter-log
      path: /tmp/gh-aw/

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

Copilot AI changed the title [WIP] Fix super linter configuration with log file Fix super-linter LOG_FILE configuration and remove unnecessary setup steps Nov 7, 2025
Copilot AI requested a review from pelikhan November 7, 2025 22:44
Copilot finished work on behalf of pelikhan November 7, 2025 22:44
@pelikhan pelikhan merged commit 4fcf780 into main Nov 7, 2025
50 of 52 checks passed
@pelikhan pelikhan deleted the copilot/fix-super-linter-configuration branch November 7, 2025 22:57
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.

2 participants