Thread (1 message) 1 message, 1 author, 2024-03-02

Re: [GSOC][PATCH] userdiff: Add JavaScript function patterns

From: Junio C Hamano <hidden>
Date: 2024-03-02 17:13:54

Christian Couder [off-list ref] writes:
quoted
+       # check results
+       test_i18ngrep "function myFunction() { return true; }" output &&
+       test_i18ngrep "function myFunction() { return false; }" output &&
+       test_i18ngrep "var myVar = function() { return false; }" output &&
+       test_i18ngrep "var myVar = function() { return true; }" output
I think we try to use just test_grep instead of test_i18ngrep these days.
Thanks for reminding.  I am tempted to suggest doing this.

------ >8 ----------- >8 ----------- >8 ----------- >8 ------
Subject: test_i18ngrep: hard deprecate and forbid its use

Since v2.44.0-rc0~109 (Merge branch 'sp/test-i18ngrep', 2023-12-27)
none of the tests we have, either in 'master' or in flight and
collected in 'seen', use test_i18ngrep.

Perhaps it is good time to update test_i18ngrep to BUG to avoid
people adding new calls to it.

Signed-off-by: Junio C Hamano <redacted>
---
 t/test-lib-functions.sh | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git c/t/test-lib-functions.sh w/t/test-lib-functions.sh
index b5eaf7fdc1..6eaf116346 100644
--- c/t/test-lib-functions.sh
+++ w/t/test-lib-functions.sh
@@ -1263,9 +1263,8 @@ test_cmp_bin () {
 	cmp "$@"
 }
 
-# Deprecated - do not use this in new code
 test_i18ngrep () {
-	test_grep "$@"
+	BUG "do not use test_i18ngrep---use test_grep instead"
 }
 
 test_grep () {
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help