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

Add bun support #481

Merged
merged 7 commits into from
Feb 10, 2025
Merged

Add bun support #481

merged 7 commits into from
Feb 10, 2025

Conversation

Songmu
Copy link
Contributor

@Songmu Songmu commented Feb 10, 2025

Pull Request detail

個人の執筆プロジェクトでBunを使っているため対応して下さると嬉しいです。

Fix or Add/Remove in this PR/このプルリクエストでやったこと

  • Bunへの対応
    • package_managerbun を指定できるように
    • bun installbunx を利用するように

Do not fix or add/remove this PR/このプルリクエストでやらなかったこと

  • DenoなどのBun以外の処理系のサポート

Pros and Cons/メリットとデメリット

Pros

  • Bunを利用する事によるCIの高速化

Cons

  • サポートするパッケージマネージャーやランタイムが増える事による、メンテナビリティの低下

Test/動作確認

  • forkした Songmu/action-textlint@master での動作確認

@tsuyoshicho
Copy link
Owner

確認して、対応を入れたいと思います。

@tsuyoshicho tsuyoshicho force-pushed the bun branch 2 times, most recently from 60aaf97 to d705160 Compare February 10, 2025 14:01
@tsuyoshicho
Copy link
Owner

npm/yarn/pnpmのテストを用意した上で、bunのテストも追加して、PASS

@tsuyoshicho tsuyoshicho added the bump:minor Bump up minor version label Feb 10, 2025
@tsuyoshicho tsuyoshicho changed the title Bunサポート Add bun support Feb 10, 2025
@tsuyoshicho tsuyoshicho merged commit acab4b7 into tsuyoshicho:master Feb 10, 2025
9 checks passed
@Songmu
Copy link
Contributor Author

Songmu commented Feb 10, 2025

丁寧に取り込んでくださってありがとうございます!活用させてもらいます!

@tsuyoshicho
Copy link
Owner

@Songmu こちらこそ、bunの対応ありがとうございます

@Songmu Songmu deleted the bun branch February 10, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump:minor Bump up minor version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants