Skip to content

remove lint for /js/ folder - #26984

Merged
prathikr merged 1 commit into
mainfrom
prathikrao/js-lint-warn-1
Jan 13, 2026
Merged

remove lint for /js/ folder#26984
prathikr merged 1 commit into
mainfrom
prathikrao/js-lint-warn-1

Conversation

@prathikr

@prathikr prathikr commented Jan 12, 2026

Copy link
Copy Markdown
Contributor

Description

js/ linting is already done in the Web CI pipeline so removing redundant check as its signaling false-positives

@guschmue guschmue 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.

lots of code left in /js - we don't want to remove lint for /js

@prathikr
prathikr enabled auto-merge (squash) January 13, 2026 00:00
@prathikr
prathikr merged commit f86a0ed into main Jan 13, 2026
101 of 102 checks passed
@prathikr
prathikr deleted the prathikrao/js-lint-warn-1 branch January 13, 2026 07:23
umangb-09 pushed a commit to umangb-09/onnxruntime that referenced this pull request Jan 13, 2026
### Description
<!-- Describe your changes. -->

`js/` linting is already done in the Web CI pipeline so removing
redundant check as its signaling false-positives

Co-authored-by: Prathik Rao <prathikrao@microsoft.com>
alex-spacemit pushed a commit to spacemit-com/onnxruntime that referenced this pull request Jan 20, 2026
### Description
<!-- Describe your changes. -->

`js/` linting is already done in the Web CI pipeline so removing
redundant check as its signaling false-positives

Co-authored-by: Prathik Rao <prathikrao@microsoft.com>
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