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

♻️ Migrate CLI print methods to single module

未关闭
#601 4 条评论 0 个 reaction 已指派 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 摘要。