apache/apisix

Merge `extra_init_by_lua_start ` and `extra_init_by_lua ` directive in test cases

オープン

#7,990 opened on 2022/09/26

GitHub で見るコントリビューターガイド
 (14 件のコメント) (0 件のリアクション) (0 人の担当者)Lua (2,860 件のフォーク)batch import
good first issuetech debt

Repository metrics

Stars
 (16,597 個のスター)
PR merge metrics
 (PR metrics pending)

説明

Discussed in https://github.com/apache/apisix/discussions/7988

Originally posted by monkeyDluffy6017 September 26, 2022 Maybe we should merge extra_init_by_lua_start and extra_init_by_lua directives in test cases, they are just different on position.

コントリビューターガイド

調査方針
背景については、https://github.com/apache/apisix/discussions/7988 の議論を確認してください。`extra init by lua start` と `extra init by lua` の両方のディレクティブを含むすべてのテストファイルを特定し、適切な場合にはそれらを単一のディレクティブに統合してください。変更後もテストカバレッジが維持されることを確認してください。
技術スタック
lua
領域
backend
Issue 種別
テスト
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
古い
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
GitLua
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
75