The GitHub Actions job "Required Checks" on texera.git/chore/monaco-lsp-v10 has 
failed.
Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).

Head commit for run:
018188d66174020db6a9f9879f8c5abc0046354c / Xinyuan Lin <[email protected]>
chore(frontend): second cleanup pass on PR-touched files

Another sweep for things that didn't quite pull their weight:

* `code-editor.component.ts`:
  * Drop the `apiWrapperStarted` static boolean; `apiWrapperStartPromise`
    is now the single source of truth — once it resolves, every later
    `await` returns immediately, and the failure branch already clears
    it back to `undefined` so a retry runs the full start sequence
    again. The boolean was just a sync fast-path masking the same
    semantic.
  * Simplify `getFileSuffixByLanguage`: the `javascript` case was
    unreachable (the constructor only ever calls
    `setLanguage("python" | "java")`, and R UDF support was retired
    earlier in this branch). Collapse to a one-line ternary with a
    `.py` default.

* `editor.worker.ts`: trim the canonical-rationale comment from 7 lines
  to 3. The full reasoning is already documented in the
  `monacoWorkerFactory` block in code-editor.component.ts; the worker
  file only needs a pointer.

* `jsdom-svg-polyfill.ts`: convert the remaining `/** ... */` block
  comments (CSSStyleSheet, CSS namespace, matchMedia, WebSocket,
  ngZorro icon suppression) to leading `//` comments to match the style
  of the SVG / loader-hook / requestIdleCallback sections that already
  use them. The mixed style was visually noisy in a long polyfill file.
  Same content, just one comment shape throughout.

No behaviour change.
yarn build:ci EXIT 0; check_binary_deps.py OK: 113; yarn test 63 / 269.

Report URL: https://github.com/apache/texera/actions/runs/25597291961

With regards,
GitHub Actions via GitBox

Reply via email to