Skip to content

Conversation

@chvmvd
Copy link
Contributor

@chvmvd chvmvd commented Jul 13, 2025

  • パッケージをアップデートしました。パッケージのアップデートによる影響はなさそうでした。
  • .gitignoreで一部がnode_modulesなどになっていたため、/node_modulesの方に統一しました。(もともとは、/node_modulesに統一されていましたが、解答例を追加しました。 #63 の時に追加されたものがnode_modulesの表記になっていたことが主な原因と思われます。)
  • サンプルプログラムにおいて、不要な半角スペースが除去されていなかったものを除去しました。(和欧文間の半角スペースを削除 #799 での修正漏れです。)
  • Viteがデフォルトで生成するファイルが変更されていたため、それに合わせてアップデートしました。

@cloudflare-workers-and-pages
Copy link

Deploying utcode-learn with  Cloudflare Pages  Cloudflare Pages

Latest commit: ff86570
Status: ✅  Deploy successful!
Preview URL: https://07da577a.utcode-learn.pages.dev
Branch Preview URL: https://update-samples.utcode-learn.pages.dev

View logs

@@ -1,14 +1,14 @@
<!doctype html>
<html lang="ja">
<head>
<meta charset="utf-8" />
<meta charset="UTF-8" />
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Viteがデフォルトで生成するファイルに合わせた形です。

<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>Vite App</title>
</head>
<body>
<div>
<canvas id="myChart"></canvas>
</div>
<script type="module" src="/main.js"></script>
<script type="module" src="/src/main.js"></script>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Viteがデフォルトで生成するファイルに合わせた形です。

@@ -2,13 +2,16 @@
"name": "chartjs",
"private": true,
"version": "0.0.0",
"type": "module",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Viteがデフォルトで生成するファイルに合わせた形です。

Comment on lines +7 to +9
"dev": "vite",
"build": "vite build",
"preview": "vite preview"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Viteがデフォルトで生成するファイルに合わせた形です。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Viteがデフォルトで生成するファイルから変更は加えていません。

Comment on lines +8 to +27
"build": "tsc -b && vite build",
"lint": "eslint .",
"preview": "vite preview"
},
"dependencies": {
"react": "^18.3.1",
"react-dom": "^18.3.1"
"react": "^19.1.0",
"react-dom": "^19.1.0"
},
"devDependencies": {
"@types/react": "^18.3.3",
"@types/react-dom": "^18.3.0",
"@vitejs/plugin-react": "^4.3.1",
"typescript": "^5.5.4",
"vite": "^5.4.1"
"@eslint/js": "^9.30.1",
"@types/react": "^19.1.8",
"@types/react-dom": "^19.1.6",
"@vitejs/plugin-react": "^4.6.0",
"eslint": "^9.30.1",
"eslint-plugin-react-hooks": "^5.2.0",
"eslint-plugin-react-refresh": "^0.4.20",
"globals": "^16.3.0",
"typescript": "~5.8.3",
"typescript-eslint": "^8.35.1",
"vite": "^7.0.4"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Viteがデフォルトで生成するファイルから変更は加えていません。

Comment on lines +1 to 9
import { StrictMode } from "react";
import { createRoot } from "react-dom/client";
import App from "./App.tsx";

ReactDOM.createRoot(document.getElementById("root") as HTMLElement).render(
<React.StrictMode>
createRoot(document.getElementById("root")!).render(
<StrictMode>
<App />
</React.StrictMode>,
</StrictMode>,
);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Viteがデフォルトで生成するファイルから変更は加えていません。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Viteがデフォルトで生成するファイルから変更は加えていません。

@chvmvd chvmvd requested a review from chelproc July 13, 2025 04:45
Copy link
Contributor

@chelproc chelproc left a comment

Choose a reason for hiding this comment

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

全部は見切れないですが多分大丈夫・・・?
サンプルプログラムのパッケージのメンテがしんどいので何か良き方法があるといいですね👀

@chvmvd chvmvd merged commit fb24ae7 into main Jul 14, 2025
3 checks passed
@chvmvd chvmvd deleted the update-samples branch July 14, 2025 12:41
@chvmvd chvmvd mentioned this pull request Aug 9, 2025
4 tasks
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.

3 participants