Skip to content

fix: remove unused vars in pipeline.behavior.test.ts - #605

Merged
kojiwakayama merged 1 commit into
mainfrom
fix/lint-pipeline-behavior-test
Mar 10, 2026
Merged

fix: remove unused vars in pipeline.behavior.test.ts#605
kojiwakayama merged 1 commit into
mainfrom
fix/lint-pipeline-behavior-test

Conversation

@kwakayama

Copy link
Copy Markdown
Contributor

Fixes lint errors breaking CI on main (#604).

  • Remove unused slug, projectId, generatedCss variables
  • Change let to const for candidatesReceived (single assignment)

CI run that's failing: https://github.com/veryfront/veryfront-code/actions/runs/22907107438

Fixes lint errors breaking CI on main:
- Remove unused slug, projectId, generatedCss variables
- Change let to const for candidatesReceived (single assignment)

Closes #604
Copilot AI review requested due to automatic review settings March 10, 2026 15:26

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 fixes CI lint failures on main by removing unused variables and applying prefer-const in an orchestrator behavior test.

Changes:

  • Removed unused slug, projectId, and generatedCss variables in pipeline.behavior.test.ts.
  • Replaced let candidatesReceived with a const assignment to satisfy prefer-const.

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

@kojiwakayama
kojiwakayama merged commit 2ebef70 into main Mar 10, 2026
18 checks passed
@kojiwakayama
kojiwakayama deleted the fix/lint-pipeline-behavior-test branch March 10, 2026 16:20
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