07th-mod / 07th-mod/python-patcher

Missing sharedassets0.assets file in language patch does not terminate install

オープン 初心者向け
#193 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
214
フォーク
12
PR マージ指標
30日以内にマージされた PR はありません

説明

Currently, if a matching `sharedassets0.assets` file is not found in the language patch, the installer will just hope for the best and use the default one.

It may be a better idea to stop the install with an error if this happens.

I think the way it works currently is that we extract the whole language patch archive directly, which automatically overwrites the sharedassets0.assets. But it may be better to leave the install as-is, without the language patch installed, if we know for sure it won't work.

----

Also see #192 if this is still to be treated as a warning, not an error

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

言語パッチアーカイブを抽出し、sharedassets0.assetsファイルを処理するコードを探してください。インストーラーにはこのファイルをチェックする関数がある可能性があります。ファイルが欠落している場合にエラーを発生させるか、インストールを停止するように変更してください。警告とエラーの違いに関する背景については、関連するissue #192を確認してください。sharedassets0.assetsファイルが欠落している言語パッチをシミュレートしてテストします。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, python
領域
cli, tooling
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
65/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。