server: entrypoints `run_*.py` are written as scripts
メンテナーはふだん 4 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 74/100
調査の方向性
Start by reading server/app/services/run_repository.py, run_registry.py, and run_discovery.py to identify the code currently executed at import time. Move each entrypoint's setup into a main() function and only assign application when the script is run directly. Done when importing these modules no longer executes the setup, while direct execution still creates the WSGI application.
索引モデルが issue の本文から書いたものです。
説明
To create the Python WSGI application, the docker images execute a script from server/app/services (run_repository.py, run_registry.py, run_discovery.py) which sets the global application variable.
These script's code is written directly to the file and is not wrapped inside functions or a if __name__ == "__main__" guard. Therefore this code can only and will always be executed when the script is imported (e.g. import app.services.run_repositroy). This complicates unit testing unnecessary, because it requires re-importing the script over and over again.
We should wrap the code that lives at file level today inside a main() function and only execute it, when the script is run directly. This could look equivalent to this:
def main() -> WSGIApp:
...
if __name__ == "__main__":
application = main()
- 主要言語
- Python
- スター
- 104
- フォーク
- 56
- 平均マージ
- 12日 13時間
- マージ済み PR（30日）
- 5
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
eclipse-basyx/basyx-python-sdk のほかの issue
-
README CI badge is broken対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンbug documentation
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
eclipse-basyx/basyx-python-sdk#656 ·
メンテナーはふだん 4 日以内に返信
-
bug server Specification
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
eclipse-basyx/basyx-python-sdk#655 ·
メンテナーはふだん 4 日以内に返信
-
bug sdk
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
eclipse-basyx/basyx-python-sdk#634 · コメント 3 件 ·
メンテナーはふだん 4 日以内に返信
-
enhancement server
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
eclipse-basyx/basyx-python-sdk#632 ·
メンテナーはふだん 4 日以内に返信
-
bug high priority
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
eclipse-basyx/basyx-python-sdk#631 ·
メンテナーはふだん 4 日以内に返信
eclipse-basyx/basyx-python-sdk の issue をすべて見る
似ている issue
-
area:core-platform effort:S priority:P3 status:in-review type:test
難易度 2/5 1時間未満 初心者へのやさしさ 80/100
randomparity/kdive#3246 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Requantification library converts m/z to mass with the hydrogen-atom mass instead of the proton massオープンbug
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
nf-core/metaboigniter#130 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
common-workflow-language/cwl-utils#477 ·
メンテナーはふだん 1 日以内に返信
-
Add dependabotオープンfeat
難易度 2/5 1時間未満 初心者へのやさしさ 72/100
open-nudge/opentemplate#204 ·
メンテナーはふだん 1 日以内に返信
-
github-stale-ci-pr-closer: KV state is double-wrapped, never persists, and hits the depth-32 limit対応中かも @VascoSch92 が今日担当しました。 オープンpriority:medium ready-for-dev
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
OpenHands/extensions#781 ·
メンテナーはふだん 1 日以内に返信