Skip to content

PR quality code checks: Disabled ESLint checks for backend#270

Merged
stasel merged 1 commit intomainfrom
disable-eslint-backend
Jan 12, 2026
Merged

PR quality code checks: Disabled ESLint checks for backend#270
stasel merged 1 commit intomainfrom
disable-eslint-backend

Conversation

@stasel
Copy link
Copy Markdown
Member

@stasel stasel commented Jan 12, 2026

It is not ready and will be re-enabled again in #248

Copilot AI review requested due to automatic review settings January 12, 2026 08:38
@HackYourFutures HackYourFutures temporarily deployed to dojo-disable-eslint-bac-rhxmnc January 12, 2026 08:38 Inactive
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

This PR temporarily disables ESLint checks for the backend by commenting out the lint-server job, with plans to re-enable it in a future PR (#248). Additionally, the PR updates GitHub Actions versions and Node.js version configuration in the lint-client job.

Changes:

  • Commented out the entire lint-server job to disable backend ESLint checks
  • Updated actions/checkout from v4 to v5 in the lint-client job
  • Updated actions/setup-node from v4 to v6 in the lint-client job
  • Changed Node.js version from '22' to 'lts/*' in the lint-client job

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

Copy link
Copy Markdown
Contributor

@MBreathe MBreathe left a comment

Choose a reason for hiding this comment

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

lgtm

@stasel stasel merged commit dfda9e7 into main Jan 12, 2026
7 checks passed
@stasel stasel deleted the disable-eslint-backend branch January 12, 2026 13:48
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.

4 participants