Bump React 19, @types/react-dom, react-dom in vscode-bicep-ui; bump ESLint 10 in playground#18942
Conversation
|
Test this change out locally with the following install scripts (Action run 24104185139) VSCode
Azure CLI
|
35fde7a to
34be4c4
Compare
34be4c4 to
d1ab165
Compare
d1ab165 to
0ab1914
Compare
|
@copilot resolve the merge conflicts in this pull request |
Co-authored-by: brendandburns <5751682+brendandburns@users.noreply.github.com>
Resolved in commit |
….0 (PR #19356) Co-authored-by: brendandburns <5751682+brendandburns@users.noreply.github.com>
Done in commit |
|
@copilot please fix CI/CD tests. |
… outputs field Agent-Logs-Url: https://github.com/Azure/bicep/sessions/d1d04e38-5b61-46b7-b6a0-f70abf394040 Co-authored-by: brendandburns <5751682+brendandburns@users.noreply.github.com>
Fixed in commit Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
…SLint 10 in playground (#18942) Upgrades React ecosystem in `/src/vscode-bicep-ui` from 18→19 and ESLint in `/src/playground` from 9→10, resolving all merge conflicts with current `main`. ## vscode-bicep-ui changes - `react` / `react-dom`: `^18.3.1` → `^19.2.4` - `@types/react` / `@types/react-dom`: `^18.x` → `^19.2.14` / `^19.2.3` - React 19 TypeScript breaking changes fixed: - `useRef<T>()` → `useRef<T | undefined>(undefined)` (initial value now required) - `JSX.Element` → `ReactElement` (global `JSX` namespace removed) - DOM ref types updated to `RefObject<T | null>` ## playground changes - `eslint`: `9.39.2` → `10.2.0` - `@eslint/js`: `9.39.2` → `10.0.1` - `eslint-plugin-notice` removed (dropped in ESLint 10); replaced with inline custom copyright rule ## Test fix - `Snapshot_command_supports_managementGroup_scope`: added missing `"outputs": {}` to expected JSON — the "Add outputs to snapshot" change (#19379) introduced this field after the test was written --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: brendandburns <5751682+brendandburns@users.noreply.github.com>
…SLint 10 in playground (#18942) Upgrades React ecosystem in `/src/vscode-bicep-ui` from 18→19 and ESLint in `/src/playground` from 9→10, resolving all merge conflicts with current `main`. ## vscode-bicep-ui changes - `react` / `react-dom`: `^18.3.1` → `^19.2.4` - `@types/react` / `@types/react-dom`: `^18.x` → `^19.2.14` / `^19.2.3` - React 19 TypeScript breaking changes fixed: - `useRef<T>()` → `useRef<T | undefined>(undefined)` (initial value now required) - `JSX.Element` → `ReactElement` (global `JSX` namespace removed) - DOM ref types updated to `RefObject<T | null>` ## playground changes - `eslint`: `9.39.2` → `10.2.0` - `@eslint/js`: `9.39.2` → `10.0.1` - `eslint-plugin-notice` removed (dropped in ESLint 10); replaced with inline custom copyright rule ## Test fix - `Snapshot_command_supports_managementGroup_scope`: added missing `"outputs": {}` to expected JSON — the "Add outputs to snapshot" change (#19379) introduced this field after the test was written --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: brendandburns <5751682+brendandburns@users.noreply.github.com>
Upgrades React ecosystem in
/src/vscode-bicep-uifrom 18→19 and ESLint in/src/playgroundfrom 9→10, resolving all merge conflicts with currentmain.vscode-bicep-ui changes
react/react-dom:^18.3.1→^19.2.4@types/react/@types/react-dom:^18.x→^19.2.14/^19.2.3useRef<T>()→useRef<T | undefined>(undefined)(initial value now required)JSX.Element→ReactElement(globalJSXnamespace removed)RefObject<T | null>playground changes
eslint:9.39.2→10.2.0@eslint/js:9.39.2→10.0.1eslint-plugin-noticeremoved (dropped in ESLint 10); replaced with inline custom copyright ruleTest fix
Snapshot_command_supports_managementGroup_scope: added missing"outputs": {}to expected JSON — the "Add outputs to snapshot" change (Add outputs to snapshot #19379) introduced this field after the test was written