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

voidmatcha 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 3fc1d4dcc5 [MINOR] Format notebook lifecycle contract for Prettier 
3.9.9
3fc1d4dcc5 is described below

commit 3fc1d4dcc592d927c0d6826e9ccf1585f18a1b66
Author: YONGJAE LEE (이용재) <[email protected]>
AuthorDate: Sat Oct 10 17:06:53 2026 +0900

    [MINOR] Format notebook lifecycle contract for Prettier 3.9.9
    
    ### What is this PR for?
    
    PR #5540 passed its E2E checks with Prettier 3.8.3. Before #5540 merged, PR 
#5546 updated the Angular lockfile to Prettier 3.9.9, which formats this union 
type differently. #5540 was merged without rerunning its checks against the 
updated master, so the E2E jobs on #5559 now stop at the Prettier check before 
Playwright starts.
    
    This PR applies only the Prettier 3.9.9 formatting to the affected type 
declaration. It does not change runtime behavior.
    
    The existing Husky hook formats staged TypeScript files with locally 
installed dependencies at commit time, but it cannot detect a later base-branch 
dependency update. To prevent similar merge-order failures, maintainers could 
consider requiring checks against the latest master or using a merge queue. 
That policy change is outside this PR.
    
    ### What type of PR is it?
    
    Improvement
    
    ### Todos
    
    - [x] Format the affected file with Prettier 3.9.9
    - [x] Check the full Angular Prettier file set
    
    ### What is the Jira issue?
    
    N/A
    
    ### How should this be tested?
    
    The following check passed locally from `zeppelin-web-angular`:
    
    ```bash
    npx --yes prettier<at>3.9.9 --check '**/*.{ts,tsx,mts,js,mjs,json,css,html}'
    ```
    
    The E2E CI jobs should then proceed past the formatting step.
    
    ### Screenshots (if appropriate)
    
    N/A
    
    ### Questions:
    
    * Does the license files need to update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No
    
    Closes #5564 from voidmatcha/fix/zeppelin-notebook-lifecycle-prettier-399.
    
    Signed-off-by: YONGJAE LEE <[email protected]>
---
 .../core-contract/lifecycle/core/runtime/host-remote-contract.ts  | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git 
a/zeppelin-web-angular/e2e/core-contract/lifecycle/core/runtime/host-remote-contract.ts
 
b/zeppelin-web-angular/e2e/core-contract/lifecycle/core/runtime/host-remote-contract.ts
index a77a30a965..77f8aaf275 100644
--- 
a/zeppelin-web-angular/e2e/core-contract/lifecycle/core/runtime/host-remote-contract.ts
+++ 
b/zeppelin-web-angular/e2e/core-contract/lifecycle/core/runtime/host-remote-contract.ts
@@ -17,13 +17,7 @@ export type NotebookParagraphStatus = 'UNKNOWN' | 'READY' | 
'PENDING' | 'RUNNING
 export type NotebookParagraphResultType = 'TEXT' | 'HTML' | 'TABLE' | 'IMG' | 
'ANGULAR' | string;
 
 export type NotebookVisualizationMode =
-  | 'table'
-  | 'multiBarChart'
-  | 'pieChart'
-  | 'lineChart'
-  | 'stackedAreaChart'
-  | 'scatterChart'
-  | string;
+  'table' | 'multiBarChart' | 'pieChart' | 'lineChart' | 'stackedAreaChart' | 
'scatterChart' | string;
 
 export type NotebookParagraphResult = Readonly<{
   type: NotebookParagraphResultType;

Reply via email to