Skip to content

Disable typescript strict checks#7131

Merged
cdavalos7 merged 10 commits into
tensorflow:masterfrom
cdavalos7:feature/upgrade17-18
Jun 19, 2026
Merged

Disable typescript strict checks#7131
cdavalos7 merged 10 commits into
tensorflow:masterfrom
cdavalos7:feature/upgrade17-18

Conversation

@cdavalos7

@cdavalos7 cdavalos7 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Motivation for features / changes

An internal large change is migrating TypeScript to versions 6 and 7, which enable strict: true by default. Ideally we would clear all the resulting warnings, but tracking them down while the TB Angular migration is still in progress is impractical. For now, we disable strict mode (strict: false) and defer the cleanup.

Technical description of changes

Screenshots of UI changes (or N/A)

Detailed steps to verify changes work correctly (as executed by you)

Alternate designs / implementations considered (or N/A)

# Conflicts:
#	tensorboard/plugins/debugger_v2/tf_debugger_v2_plugin/views/common/_lib.scss
#	tensorboard/plugins/debugger_v2/tf_debugger_v2_plugin/views/execution_data/execution_data_component.scss
#	tensorboard/plugins/debugger_v2/tf_debugger_v2_plugin/views/graph/graph_op_component.scss
#	tensorboard/webapp/feature_flag/views/feature_flag_dialog_component.scss
#	tensorboard/webapp/metrics/views/card_renderer/image_card_component.scss
#	tensorboard/webapp/metrics/views/card_renderer/scalar_card_component.scss
#	tensorboard/webapp/metrics/views/card_renderer/vis_linked_time_selection_warning_component.scss
#	tensorboard/webapp/metrics/views/main_view/filter_input_component.scss
#	tensorboard/webapp/metrics/views/main_view/main_view_component.scss
#	tensorboard/webapp/metrics/views/main_view/pinned_view_component.scss
#	tensorboard/webapp/metrics/views/metrics_container.scss
#	tensorboard/webapp/metrics/views/right_pane/scalar_column_editor/scalar_column_editor_component.scss
#	tensorboard/webapp/metrics/views/right_pane/settings_view_component.scss
#	tensorboard/webapp/notification_center/_views/notification_center_component.scss
#	tensorboard/webapp/runs/views/runs_table/runs_data_table.scss
#	tensorboard/webapp/runs/views/runs_table/runs_group_menu_button_component.scss
#	tensorboard/webapp/theme/_tb_palette.scss
#	tensorboard/webapp/theme/_tb_theme.template.scss
#	tensorboard/webapp/theme/_variable.scss
#	tensorboard/webapp/widgets/card_fob/card_fob_component.scss
#	tensorboard/webapp/widgets/content_wrapping_input/content_wrapping_input_component.scss
#	tensorboard/webapp/widgets/data_table/column_selector_component.scss
#	tensorboard/webapp/widgets/data_table/context_menu_component.scss
#	tensorboard/webapp/widgets/data_table/data_table_component.scss
#	tensorboard/webapp/widgets/data_table/filter_dialog_component.scss
#	tensorboard/webapp/widgets/data_table/header_cell_component.scss
#	tensorboard/webapp/widgets/dropdown/dropdown_component.scss
#	tensorboard/webapp/widgets/line_chart_v2/sub_view/line_chart_axis_view.scss
@cdavalos7 cdavalos7 requested a review from arcra June 18, 2026 21:08
@cdavalos7 cdavalos7 merged commit 37d609f into tensorflow:master Jun 19, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants