Make a central `PVValue` definition
まだ誰も着手していません。
評価
調査の方向性
Genie Python と ibex-non-ca-helpers にあるものも含め、PVValue のすべての定義と import を特定します。まず現在の定義とその利用箇所を把握します。定義を ibex-non-ca-helpers に集約し、他の定義と import を置き換えた後、現在のすべての利用箇所が集約先から解決されることを確認します。
索引モデルが issue の本文から書いたものです。
説明
Feature Description
As a developer, I would like to be able to import PVValue from a central location anywhere in the project without needing to import Genie Python.
Explanation
Genie python is a large import, and so in some cases it has been avoided to import when just the PVValue type is required, this has lead to multiple definitions of PVValue in the project, these should be removed and replaced with a central definition in ibex-non-ca-helpers so that if needed the type can be modified in a single location, and code duplication is avoided.
Additional Information
As well as the changes in genie python and other locations where PVValue is defined, all imports of PVValue will need to be pointed at the new location.
Acceptance Criteria
-
PVValueis defined in a central location where it can easily be imported for all uses. - All current definitions of
PVValueand imports of them are replaced with an import of this central definition.
How to Review
Before making a PR...
- Provide verbose instructions for the reviewer to test your changes
- Describe how you have implemented testing for this feature
- Provide screenshots of the feature to help the reviewer if relevant
If not applicable, write "Not applicable"
...
To the reviewer: Make sure to update submodules!
- 主要言語
- 言語のデータがありません
- スター
- 6
- フォーク
- 2
- 平均マージ
- 12日 19時間
- マージ済み PR（30日）
- 3
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ISISComputingGroup/IBEX のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
ISISComputingGroup/IBEX#9023 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
ISISComputingGroup/IBEX#9005 ·
-
Good First Issue
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
ISISComputingGroup/IBEX#8889 ·
-
0
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
ISISComputingGroup/IBEX#2304 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
ISISComputingGroup/IBEX#2078 ·
ISISComputingGroup/IBEX の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
acsandmann/rift#577 ·
メンテナーはふだん 1 日以内に返信
-
type: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
blamechris/chroxy#8579 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
-
needs triage
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
darnitdevorg/darnit#587 ·
メンテナーはふだん 1 日以内に返信