apache/apisix

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

Open

#7.990 geöffnet am 26. Sept. 2022

Auf GitHub ansehen
 (14 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Lua (2.860 Forks)batch import
good first issuetech debt

Repository-Metriken

Stars
 (16.597 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 8h) (109 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Überprüfen Sie die Diskussion unter https://github.com/apache/apisix/discussions/7988 für den Kontext. Identifizieren Sie alle Testdateien, die sowohl die Direktiven `extra init by lua start` als auch `extra init by lua` enthalten, und führen Sie sie, wo angemessen, in eine einzelne Direktive zusammen. Stellen Sie sicher, dass die Änderung die Testabdeckung beibehält.
Tech Stack
lua
Domain
backend
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitLua
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75