iLPdev / iLPdev/discMapper

Obsolete functions or functions that can be simplified.

オープン
#130 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
Lua
スター
8
フォーク
5
PR マージ指標
30日以内にマージされた PR はありません

説明

Merge Rooms : No longer necessary, since the hash from gmcp.room.info positively identifies a room, no need to merge rooms, since different rooms will have different hashes and thus different room ID's.

function set_room : At the end of this functions, an onMoveMap event is raised... yet you have no handler to handle the event. Is this an event handled by the mapper that I haven't found?

function check_room(...) : Since rooms are positively identified by a hash, why do you check against room name and exits? Big issue I have with this, is that some rooms have exits that are not obvious. Once I use said exit, the exit is created, and thus this function no longer identifies a perfectly valid room and won't show you being in that room.

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

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

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず Merge Rooms、set_room、check_room 関数を見つけ、次に onMoveMap イベントのハンドラーまたは参照を検索します。gmcp.room.info のハッシュがどのように部屋を識別し、出口がどのように表現されているかを確認します。obsolete なマージとチェックが削除または簡略化され、有効な部屋の認識が妨げられない状態になれば完了です。

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

評価

技術スタック
lua
領域
tooling
issue の種類
リファクタリング
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
28/100

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

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