Election-Tech-Initiative / Election-Tech-Initiative/electionguard-python

♻️ Migrate CLI print methods to single module

オープン
#601 コメント 4 件 リアクション 0 件 担当者 4 名 @Alopezjr2002 が担当を希望しています GitHub で見る
enhancement good first issue hacktoberfest help wanted
主要言語
Python
スター
168
フォーク
103
PR マージ指標
30日以内にマージされた PR はありません

説明

### Is there an existing issue for this?

- [X] I have searched the existing issues

### Suggestion

To improve the CLI and be a bit more pythonic, migrate the common click to echo to a more aptly named module. Ideally, this should contain the references to `click.echo` to a single module, but the closest to this is best.

### Possible Implementation

Create a `print_utils.py` file in the `electionguard_cli` to be used for all print functionality. These should all be methods and no classes should be necessary. This should allow the deletion of `e2estepbase.py`

Suggested items to migrate:
https://github.com/microsoft/electionguard-python/blob/03698c683ab35f608e35dc9b66217caa1d40f7eb/src/electionguard_cli/e2e_steps/input_retrieval_step.py#L42
https://github.com/microsoft/electionguard-python/blob/03698c683ab35f608e35dc9b66217caa1d40f7eb/src/electionguard_cli/e2e_steps/e2e_step_base.py#L14
https://github.com/microsoft/electionguard-python/blob/03698c683ab35f608e35dc9b66217caa1d40f7eb/src/electionguard_cli/e2e_steps/e2e_step_base.py#L19

https://github.com/microsoft/electionguard-python/blob/03698c683ab35f608e35dc9b66217caa1d40f7eb/src/electionguard_cli/e2e_steps/print_results_step.py#L13

### Anything else?

_No response_

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

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

評価

この issue はまだ評価されていません。

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

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