no linting in ci

[bmschmidt]

Remove requirement for linting in CI, because we don't pass all the lint rules in this repo yet.

---

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.

^{This description was created by for 025f8f8. You can customize this summary. It will automatically update as commits are pushed.}

[bmschmidt]

• no linting in ci #194 : 2 dependent PRs (#81 , #192 ) 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed everything up to 025f8f8 in 29 seconds. Click for details.

• Reviewed 29 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. .github/workflows/ci.yml:39

• Draft comment:
  Temporary disable of ESLint in CI is acceptable, but add a TODO to re-enable when lint issues are resolved.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 70% None

Workflow ID: wflow_WhMulEoUsGulFd14

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[bmschmidt]

Merge activity

• May 26, 4:18 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• May 26, 4:18 PM UTC: @bmschmidt merged this pull request with Graphite.