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

chore: Remove @types/uuid #886

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

takejohn
Copy link
Contributor

What

uuidパッケージはバージョン11からTypeScriptに移行したので、不要となった@types/uuidを依存関係から削除します。

Why

Additional info (optional)

@takejohn takejohn changed the title Remove @types/uuid chore: Remove @types/uuid Dec 25, 2024
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

see 30 files with indirect coverage changes

@FineArchs
Copy link
Member

「Typescriptにサポートされたライブラリ」というのがよくわかっていないのですが、どういう感じなんでしょうか?

@takejohn
Copy link
Contributor Author

「Typescriptにサポートされたライブラリ」というのがよくわかっていないのですが、どういう感じなんでしょうか?

uuidパッケージは、.jsファイルと同一ディレクトリ内に同名の.d.tsファイルが配置されているため、TypeScriptが型付けできるようです。

@FineArchs
Copy link
Member

あー、なるほど!今理解しました。
今までuuid@types/uuidで別々のパッケージだったのが(uuidがTypescriptで書き換えられたため事実上)統合された感じですね。

@takejohn takejohn merged commit 5b3e5e7 into aiscript-dev:master Jan 3, 2025
4 checks passed
@takejohn takejohn deleted the chore/remove-types-uuid branch January 3, 2025 08:39
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.

4 participants