Skip to content

Commit 787011f

Browse files
authored
no linting in ci (#194)
Remove requirement for linting in CI, because we don't pass all the lint rules in this repo yet. <!-- ELLIPSIS_HIDDEN --> ---- > [!IMPORTANT] > Remove linting requirement from CI workflow due to existing lint rule failures. > > - **CI Workflow**: > - Commented out the ESLint linting step in `.github/workflows/ci.yml`. > - Linting is not required in CI due to existing lint rule failures. > - TypeScript checking and test execution steps remain active. > > <sup>This description was created by </sup>[<img alt="Ellipsis" src="https://img.shields.io/badge/Ellipsis-blue?color=175173">](https://www.ellipsis.dev?ref=nomic-ai%2Fdeepscatter&utm_source=github&utm_medium=referral)<sup> for 025f8f8. You can [customize](https://app.ellipsis.dev/nomic-ai/settings/summaries) this summary. It will automatically update as commits are pushed.</sup> <!-- ELLIPSIS_HIDDEN -->
1 parent f6ea3bc commit 787011f

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

.github/workflows/ci.yml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -36,16 +36,16 @@ jobs:
3636
echo "Files to lint:"
3737
cat ~/lintable_files
3838
39-
- name: Lint
40-
run: |
41-
if [ -s ~/lintable_files ]; then
42-
npx eslint $(cat ~/lintable_files)
43-
else
44-
echo "No files to lint"
45-
fi
46-
39+
# - name: Lint
40+
# run: |
41+
# if [ -s ~/lintable_files ]; then
42+
# npx eslint $(cat ~/lintable_files)
43+
# else
44+
# echo "No files to lint"
45+
# fi
4746
- name: TypeScript Checker
4847
run: npx tsc --emitDeclarationOnly
4948

5049
- name: Run Tests
5150
run: npm run test
51+

0 commit comments

Comments
 (0)