Re-evaluate performances for []*Struct over []Struct
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- go
調査の方向性
まず、pkg/test/fetcher.go、pkg/job/reporter.go、pkg/job/job.go、pkg/runner/job_runner.go、pkg/jobmanager/jobmanager.go、および URI と literal の test fetcher にある、一覧に記載された使用箇所を読みます。Targets を含む []Struct と []*Struct のケースを比較し、それぞれの表現を一貫して評価します。すべての一覧に記載されたケースを再評価し、正当な変更を影響を受けるファイル全体に適用すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Wednesday Feb 05, 2020 at 12:48 GMT
Originally opened as https://github.com/facebookincubator/contest/issues/21
Using []Struct is in most cases probably desirable, but in some cases []*Struct might make sense (e.g. Targets). We should re-evaluate these cases. Here is where we use []*, excluding Targets:
pkg/test/fetcher.go:16: Fetch(interface{}) (string, []*TestStepDescriptor, error)
pkg/job/reporter.go:20: Report(cancel <-chan struct{}, parameters interface{}, results []*test.TestResult, ev testevent.Fetcher) (bool, interface{}, error)
pkg/job/job.go:27: TestDescriptors []*test.TestDescriptor
pkg/job/job.go:63: Tests []*test.Test
pkg/runner/job_runner.go:49: testResults []*test.TestResult
pkg/jobmanager/jobmanager.go:88: tests := make([]*test.Test, 0, len(jd.TestDescriptors))
plugins/testfetchers/uri/uri.go:86:func (tf *URI) Fetch(params interface{}) (string, []*test.TestStepDescriptor, error) {
plugins/testfetchers/uri/uri.go:117: Steps []*test.TestStepDescriptor
plugins/testfetchers/literal/literal.go:29: Steps []*test.TestStepDescriptor
plugins/testfetchers/literal/literal.go:54:func (tf *Literal) Fetch(params interface{}) (string, []*test.TestStepDescriptor, error)
- 主要言語
- Go
- スター
- 20
- フォーク
- 17
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
linuxboot/contest のほかの issue
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
-
update readme対応中かも @mimir-d が 1253 日前に担当しました。 オープンbug documentation
-
enhancement good first issue
難易度 3/5 1〜2日 初心者へのやさしさ 38/100
-
enhancement good first issue v2_overhaul
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
-
enhancement good first issue
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
linuxboot/contest の issue をすべて見る
似ている issue
-
bug triage
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
FairwindsOps/nova#484 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
automated-analysis code-quality cookie
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
github/gh-aw#67517 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
[otelcol] print-config help text still requires the removed otelcol.printInitialConfig feature gate対応中かも @girishkvs が今日担当しました。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
open-telemetry/opentelemetry-collector#16143 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug good first issue load-balancing
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
ktrubilo9/edge-proxy#53 ·