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

tbonelee pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zeppelin.git


The following commit(s) were added to refs/heads/master by this push:
     new 9b8d82beb9 [ZEPPELIN-6635] Remove unnecessary git add from lint-staged 
configuration
9b8d82beb9 is described below

commit 9b8d82beb9eb884568fad896ee8f5b28a4c716f6
Author: Minho Jang <[email protected]>
AuthorDate: Sun Aug 9 22:36:06 2026 +0900

    [ZEPPELIN-6635] Remove unnecessary git add from lint-staged configuration
    
    ### What is this PR for?
    This PR removes unnecessary `git add` commands from the `lint-staged` 
configuration in `zeppelin-web-angular/package.json`.
    
    The project currently uses `lint-staged` v15, and lint-staged has 
automatically re-staged task modifications since v10. Keeping explicit `git 
add` entries is outdated and produce warnings, so this cleanup leaves the 
existing ESLint and Prettier tasks unchanged while relying on lint-staged's 
built-in staging behavior.
    
    ### What type of PR is it?
    Improvement
    
    ### What is the Jira issue?
    * https://issues.apache.org/jira/browse/ZEPPELIN-6635
    
    ### How should this be tested?
      * Run the frontend pre-commit workflow with staged 
TypeScript/JavaScript/JSON/CSS/HTML changes and confirm `lint-staged` still 
runs ESLint/Prettier successfully.
      * Confirm no lint-staged warning is emitted for using `git add` in task 
configuration.
    
    ### Questions:
    * Does the license files need to update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No
    
    
    Closes #5402 from miinhho/remove-git-add-from-lint-staged.
    
    Signed-off-by: ChanHo Lee <[email protected]>
---
 zeppelin-web-angular/package.json | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/zeppelin-web-angular/package.json 
b/zeppelin-web-angular/package.json
index 756dab3127..48421f7b21 100644
--- a/zeppelin-web-angular/package.json
+++ b/zeppelin-web-angular/package.json
@@ -118,12 +118,10 @@
   "lint-staged": {
     "**/*.ts": [
       "cross-env NODE_OPTIONS='--max-old-space-size=8192' eslint --fix",
-      "./node_modules/.bin/prettier --write",
-      "git add"
+      "./node_modules/.bin/prettier --write"
     ],
     "**/*.{js,json,css,html}": [
-      "./node_modules/.bin/prettier --write",
-      "git add"
+      "./node_modules/.bin/prettier --write"
     ]
   }
 }

Reply via email to