This is an automated email from the ASF dual-hosted git repository.

aglinxinyuan pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/texera.git


The following commit(s) were added to refs/heads/main by this push:
     new 6d0ff81bbe chore(dependencies): drop `python-lsp-server` (#4474)
6d0ff81bbe is described below

commit 6d0ff81bbe7562547fdc282a0911325bb95ad871
Author: Jiadong Bai <[email protected]>
AuthorDate: Wed Apr 22 16:34:36 2026 -0700

    chore(dependencies): drop `python-lsp-server` (#4474)
    
    ### What changes were proposed in this PR?
    
    Drop `python-lsp-server` from Texera's release dependencies, per the
    discussion on #4467.
    
    `python-lsp-server` pulled in five ASF Category X packages:
    
    - `pylint` (GPL-2.0)
    - `astroid` (LGPL-2.1)
    - `rope` (LGPL-3.0)
    - `pytoolconfig` (LGPL-3.0)
    - `docstring-to-markdown` (LGPL-2.1)
    
    Usage has been minimal; users who need a Python LSP can install one
    (`pylsp`, `pyright`, etc.) in their own environment.
    
    Two places install it and both are removed:
    
    - `amber/requirements.txt` — drops the `python-lsp-server[all]==1.12.0`
    and `python-lsp-server[websockets]==1.12.0` lines.
    - `bin/computing-unit-worker.dockerfile` — drops the inline `pip3
    install python-lsp-server ...` step that runs before the
    `requirements.txt` install.
    
    ### Any related issues, documentation, discussions?
    
    Closes #4456.
    Closes #4457.
    Closes #4458.
    Closes #4459.
    Closes #4460.
    Closes #4467.
    
    ### How was this PR tested?
    
    After removing the two install locations, a fresh `pip install -r
    amber/requirements.txt` no longer installs any of the five Category X
    packages. The worker image build no longer pulls them either.
    
    ### Was this PR authored or co-authored using generative AI tooling?
    
    Generated-by: Claude Code (Claude Opus 4.7)
    
    Co-authored-by: Claude Opus 4.7 (1M context) <[email protected]>
    Co-authored-by: Xinyuan Lin <[email protected]>
---
 amber/requirements.txt               | 2 --
 bin/computing-unit-worker.dockerfile | 1 -
 2 files changed, 3 deletions(-)

diff --git a/amber/requirements.txt b/amber/requirements.txt
index b545fd9654..562249e9ba 100644
--- a/amber/requirements.txt
+++ b/amber/requirements.txt
@@ -37,8 +37,6 @@ dataclasses==0.6
 Deprecated==1.2.14
 fs==2.4.16
 praw==7.6.1
-python-lsp-server[all]==1.12.0
-python-lsp-server[websockets]==1.12.0
 bidict==0.22.0
 cached_property==1.5.2
 psutil==5.9.0
diff --git a/bin/computing-unit-worker.dockerfile 
b/bin/computing-unit-worker.dockerfile
index b0a1b9f2af..620719a3c3 100644
--- a/bin/computing-unit-worker.dockerfile
+++ b/bin/computing-unit-worker.dockerfile
@@ -58,7 +58,6 @@ RUN apt-get update && apt-get install -y \
 
 # Install Python packages
 RUN pip3 install --upgrade pip setuptools wheel && \
-    pip3 install python-lsp-server python-lsp-server[websockets] && \
     pip3 install -r /tmp/requirements.txt && \
     (pip3 install --no-cache-dir --find-links https://pypi.org/simple/ -r 
/tmp/operator-requirements.txt || \
      pip3 install --no-cache-dir wordcloud==1.9.2)

Reply via email to