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

AtsEXを1.0.0-RC5に更新 #12

Merged
merged 3 commits into from
Jan 13, 2024

Conversation

TetsuOtter
Copy link
Contributor

0.19.0 (beta版) を参照していることによるエラーが発生している例を見かけたため、使用するパッケージバージョンを 1.0.0-RC5 (現在の最新RC版) に更新しました。

なお、#11 の変更を含んでいるため、そちらがmergeされ次第こちらをrebase & force pushします。


相談したいこと

動作環境について

以下の状態で読み込みができることを確認しましたが、READMEにどう記載すべきか迷い、とりあえず何も変更していません。

  • AtsEX ver1.0.0-RC5 (v1.0.40101.1)
  • Windows11 (22H2)
  • BVE Trainsim version 6.0.7554.619

お手元で確認頂いてそちらの環境情報を書くか、私の環境情報で書き換えてしまうか、両方書いてしまうか、どれにしましょう?

依存環境について

依存パッケージを更新したため、依存環境の記述も書き換える必要があります。
こちらについて、現在間接的に依存しているパッケージも記載されていますが、直接依存しているパッケージを書くだけでもいいのではないかと思っています。
関節依存まで詳しく知りたい方には、packages.lock.json を見るよう案内する形になるかと思います。

@TetsuOtter TetsuOtter marked this pull request as ready for review January 13, 2024 14:17
@stop-pattern
Copy link
Owner

stop-pattern commented Jan 13, 2024

いろいろ対応ありがとうございます!

#11 について

問題なさそうだったのでマージしました
rebase & push -f お願いします 🙇
これ書いてる間にpushされてましたね、ありがとうございます

相談

動作環境について

  • atsexは最新版にしましょう
    • 古いのを使っててもいいことないです
    • こっちの環境では試してないですがそっちの環境で問題なければreadme更新していいと思います
  • win11については追記でいいと思います
  • bveは同じなので変更なし

依存環境について

正直間接依存を書かなくてもいいのはそうだと思うので消してもいいとは思います
でも書いてあって困ることもないので消さなくてもいいとも思いますが、消すとちょっとすっきりするので消した方がいいかなぁ?

お願い

次の項目についてreadmeの更新をお願いできますか?
それも含めてマージしたいです 🙏

  • AtsEXを1.0.0-RC5に変更
  • win11追記
  • 依存関係をいい感じに
    • ここの判断はお任せします

@TetsuOtter
Copy link
Contributor Author

README更新、かしこまりました!

依存関係については、間接参照の部分を削除したうえで、「packages.lock.json」に全部書いてあるよ的なことを書こうと思います。

- AtsEXを1.0.0-RC5に更新
- 対応Winに11を追加
- 依存関係の記載を直接参照だけに変更
README.md Outdated Show resolved Hide resolved
@stop-pattern
Copy link
Owner

更新ありがとうございました 🙇
問題なさそうなのでマージしちゃいますね

@stop-pattern stop-pattern merged commit ff3045e into stop-pattern:main Jan 13, 2024
1 check passed
@TetsuOtter TetsuOtter deleted the feature/atsex-rc5 branch January 13, 2024 15:55
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