tbonelee opened a new pull request, #4989: URL: https://github.com/apache/zeppelin/pull/4989
### What is this PR for? This PR enables `strict: true` in the frontend Typescript configuration (`tsconfig.json`) to enhance type safety and catch potential issues at compile time. During this migration: - Existing logic was preserved as much as possible. - `non-null assertion` was avoided where possible. In cases whre Angular lifecycle constraints (e.g., `ngOnInit, @Input`) made it unavoidable, it was used with caution. - Optional properties introduced due to potential `undefined` values now throw intended errors when accessed unsafely, making error sources more explicit and enable type narrowing. - `any` type was avoided where possible. In unavoidable cases, it was used with `tslint:disable` or `@ts-ignore` comments. This improves type robustness and aligns with modern TypeScript best practices, while minimizing risk to existing functionality. ### What type of PR is it? Improvement ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-6252 ### How should this be tested? - Ensure the frontend builds successfully. - Manual verification of major UI flows. ### Screenshots (if appropriate) ### Questions: * Does the license files need to update? No * Is there breaking changes for older versions? No breaking changes expected * Does this needs documentation? No -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
