Capturing groups - #345
Capturing groups#345EzequielCaste merged 5 commits into
Conversation
joaquinelio
left a comment
There was a problem hiding this comment.
Por perezoso no verifiquΓ© si hay version mas nueva con ingles
ni si hay mΓ‘s archivos por traducir que estos,
usualmente lo hago como parte del review
Que aburrido que es este tema.
ExtraΓ±o a @cortizg...
No lo habia leido, me hubiera venido bien lo de matchall y lo de grupo con nombre
este es mi regexp para el api issue17:
/([[\sX]])\s([.+])\s?((http[.\S]+))\s?((@\w+))?\s?(#\d+)?/
bytheway,
en este proyecto no importa para los marcados [X],
pero por esto del formato del match deberias respetar los campos,
al poner 2 autores 2 pr, o cosas ssin los identif que separan los campos, el bot se confundirΓa.
|
Please make the requested changes. After it, add a comment "/done". |
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
EzequielCaste
left a comment
There was a problem hiding this comment.
otro cambio
|
@EzequielCaste "xxxobjetos/clases/regxx con nombre" |
ok lo dejamos como Grupo con nombre aunque me gusta mΓ‘s la otra forma...queda mΓ‘s profesional |
|
Thank you π I updated the Progress Issue #17 π π π |
No description provided.