- last update
6 months ago 25,906 pass, 1,808 fail - ๐ต๐ญPhilippines abhaypai
Landed here from Bug smash initiative.
+1 Patch is applied successfully for version 11.x-dev too and starting test for 11.x-dev version
- ๐ฎ๐ณIndia Prashant.c Dharamshala
Prashant.c โ made their first commit to this issueโs fork.
- Status changed to Needs review
6 months ago 3:13pm 19 December 2023 - ๐ฎ๐ณIndia Prashant.c Dharamshala
@Anybody
Created MR against 11.x by taking the changes from #8 ๐ Removing trailing slashes from robots.txt Needs review .Thank you.
- Status changed to RTBC
6 months ago 4:18pm 19 December 2023 - ๐ฉ๐ชGermany Anybody Porta Westfalica
Thanks @Prashant.c! All tests are passing green, so I think we should get this out of the way! Marking this RTBC.
- last update
6 months ago Patch Failed to Apply - Status changed to Needs review
6 months ago 8:43pm 19 December 2023 - ๐ธ๐ฐSlovakia poker10
According to the Google robots.txt description here: https://developers.google.com/search/docs/crawling-indexing/robots/robot... , the
/search
should match any path that starts with/search
. So I do not think we need both/search
and/search/
(and similar for the second change). The rules from robots.txt should be the "starts with" rules.The second question is, we have
/admin/
in robots.txt./admin
path is valid as well, so why we are not changing this too? - Status changed to Needs work
6 months ago 9:26pm 19 December 2023 - ๐บ๐ธUnited States smustgrave
Can it be documented in the issue summary what paths were chosen and why.