Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update React, Monaco, and replace rescripts with rewired #3

Merged
merged 5 commits into from
Nov 28, 2024

Conversation

shanejonas
Copy link
Member

@shanejonas shanejonas commented Nov 28, 2024

Update logs-react, playground, and inspector packages to use the latest React, React-DOM, and Monaco Editor versions, and replace rescripts with react-app-rewired.

  • Update dependencies:

    • Update react and react-dom to the latest versions in packages/playground/package.json.
    • Update monaco-editor to the latest version in packages/playground/package.json.
  • Replace rescripts with react-app-rewired:

    • Replace @rescripts/cli and @rescripts/utilities with react-app-rewired in devDependencies of packages/inspector/package.json, packages/logs-react/package.json, and packages/playground/package.json.
    • Update start, build, and test scripts to use react-app-rewired in packages/inspector/package.json, packages/logs-react/package.json, and packages/playground/package.json.
  • Rename and update configuration files:

    • Rename packages/inspector/monaco-rescript.js to packages/inspector/config-overrides.js and update content to use MonacoWebpackPlugin.
    • Rename packages/logs-react/rescript-monaco.js to packages/logs-react/config-overrides.js and update content to use MonacoWebpackPlugin.
    • Rename packages/playground/rescript-monaco.js to packages/playground/config-overrides.js and update content to use MonacoWebpackPlugin.
  • Remove unnecessary scripts:

    • Remove turbo run commands from scripts in packages/playground/package.json.

For more details, open the Copilot Workspace session.

Update `logs-react`, `playground`, and `inspector` packages to use the latest React, React-DOM, and Monaco Editor versions, and replace `rescripts` with `react-app-rewired`.

- **Update dependencies:**
  - Update `react` and `react-dom` to the latest versions in `packages/playground/package.json`.
  - Update `monaco-editor` to the latest version in `packages/playground/package.json`.

- **Replace `rescripts` with `react-app-rewired`:**
  - Replace `@rescripts/cli` and `@rescripts/utilities` with `react-app-rewired` in `devDependencies` of `packages/inspector/package.json`, `packages/logs-react/package.json`, and `packages/playground/package.json`.
  - Update `start`, `build`, and `test` scripts to use `react-app-rewired` in `packages/inspector/package.json`, `packages/logs-react/package.json`, and `packages/playground/package.json`.

- **Rename and update configuration files:**
  - Rename `packages/inspector/monaco-rescript.js` to `packages/inspector/config-overrides.js` and update content to use `MonacoWebpackPlugin`.
  - Rename `packages/logs-react/rescript-monaco.js` to `packages/logs-react/config-overrides.js` and update content to use `MonacoWebpackPlugin`.
  - Rename `packages/playground/rescript-monaco.js` to `packages/playground/config-overrides.js` and update content to use `MonacoWebpackPlugin`.

- **Remove unnecessary scripts:**
  - Remove `turbo run` commands from `scripts` in `packages/playground/package.json`.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/open-rpc/tools?shareId=XXXX-XXXX-XXXX-XXXX).
@@ -78,7 +72,7 @@
"typescript": "^3.7.3"
},
"rescripts": [
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to remove this

shanejonas and others added 4 commits November 28, 2024 18:18
…kage.json` to use `react-app-rewired`

* **`packages/inspector/package.json`**
  - Replace `@rescripts/cli` and `@rescripts/utilities` with `react-app-rewired` in `devDependencies`
  - Update `test` script to use `react-app-rewired`
  - Remove `rescripts` entry

* **`packages/playground/package.json`**
  - Replace `@rescripts/cli` and `@rescripts/utilities` with `react-app-rewired` in `devDependencies`
  - Remove `rescripts` entry
Update `logs-react`, `playground`, and `inspector` to the latest `react`, `react-dom`, and `monaco`, and replace `rescripts` with `react-app-rewired`.

* Update `react` and `react-dom` to version `18.2.0` in `packages/logs-react/package.json`, `packages/playground/package.json`, and `packages/inspector/package.json`.
* Update `monaco-editor` to version `0.34.1` in `packages/logs-react/package.json`, `packages/playground/package.json`, and `packages/inspector/package.json`.
* Replace `rescripts` with `react-app-rewired` in `packages/logs-react/package.json`, `packages/playground/package.json`, and `packages/inspector/package.json`.
* Rename `rescript-monaco.js` to `config-overrides.js` and ensure the `MonacoWebpackPlugin` configuration is equivalent to the old `rescript-monaco.js` in `packages/playground/config-overrides.js`.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/open-rpc/tools?shareId=XXXX-XXXX-XXXX-XXXX).
@shanejonas shanejonas merged commit c7699c9 into master Nov 28, 2024
0 of 3 checks passed
@BelfordZ BelfordZ deleted the shanejonas/update-react-monaco-rewired branch November 28, 2024 23:47
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