Skip to content

ENG-1428 Add eslint check to ci#781

Merged
mdroidian merged 5 commits intomainfrom
eng-1428-add-eslint-check-to-ci
Feb 13, 2026
Merged

ENG-1428 Add eslint check to ci#781
mdroidian merged 5 commits intomainfrom
eng-1428-add-eslint-check-to-ci

Conversation

@mdroidian
Copy link
Contributor

@mdroidian mdroidian commented Feb 13, 2026

…es. Introduced a new job to lint only modified JavaScript and TypeScript files in pull requests, ensuring code quality and adherence to ESLint standards.
@linear
Copy link

linear bot commented Feb 13, 2026

@supabase
Copy link

supabase bot commented Feb 13, 2026

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

View 5 additional findings in Devin Review.

Open in Devin Review

…d a mechanism to track linting results for changed files, ensuring the CI process exits with a failure status if any linting issues are detected. Removed the unused lintWarningProbe utility file.
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 new potential issue.

View 6 additional findings in Devin Review.

Open in Devin Review

…. Implemented a dynamic fetch depth strategy to determine the merge-base SHA, allowing for better handling of long-lived branches. Added error handling for cases where the merge-base cannot be determined within the specified depth limits.
@mdroidian mdroidian merged commit 9f9a0fb into main Feb 13, 2026
6 checks passed
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.

1 participant