Refactoring core.ts
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
- #838 @BassemHalim による — マージされずにクローズ
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 25/100
- issue の種類
- リファクタリング
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- typescript
調査の方向性
まず src/livecodes/core.ts を読み、issue に記載された grep コマンドで特定された 98 個の関数を確認します。ConfigManager、EditorManager、StorageManager のようなモジュール分割を定義し、動作を変更せずに既存の関数を維持します。合意したモジュールが core.ts から抽出されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Hi,
While working on another PR I noticed that core.ts was extremely long and it was a little hard to figure out how to implement some changes so I am planning on refactoring that code into multiple modules to make it more maintainable.
For example, core.ts has 98 functions
$ grep -c "= () =>" src/livecodes/core.ts
98
So I am thinking we could extract some modules from core.ts like a ConfigManager, EditorManager, StorageManger, etc. without changing the functions and then later we can refactor the long functions.
This might be too much for a single PR so maybe break the PRs into something like extracting ConfigManger from core.ts, extracting x etc.
What do you think?
- 主要言語
- TypeScript
- スター
- 1.5k
- フォーク
- 267
- 平均マージ
- 7時間 47分
- マージ済み PR（30日）
- 14
環境構築
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
live-codes/livecodes のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
live-codes/livecodes#1079 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 50/100
live-codes/livecodes#1060 · コメント 5 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
live-codes/livecodes#1009 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
live-codes/livecodes#1004 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
I18n: Hungarianオープン
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
live-codes/livecodes#980 · コメント 6 件 ·
メンテナーはふだん 1 日以内に返信
live-codes/livecodes の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
pending close triaging
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
anomalyco/opencode#62582 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
feature request
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
callstack/react-native-paper#5189 ·
メンテナーはふだん 1 日以内に返信
-
Add: ESP 2 ROオープンapproved check:passed streams:add
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信