Skip to content

celestino ejercicio 2 #9

Open
OldSnake256 wants to merge 11 commits intogbobr:masterfrom
OldSnake256:master
Open

celestino ejercicio 2 #9
OldSnake256 wants to merge 11 commits intogbobr:masterfrom
OldSnake256:master

Conversation

@OldSnake256
Copy link

piso piso piso piso piso piso piso piso piso piso piso piso piso piso piso piso piso piso

@gbobr
Copy link
Owner

gbobr commented Oct 12, 2018

run tests

@gbobr
Copy link
Owner

gbobr commented Oct 12, 2018

Can one of the admins verify this patch?

@gbobr
Copy link
Owner

gbobr commented Oct 12, 2018

rerun tests

1 similar comment
@gbobr
Copy link
Owner

gbobr commented Oct 12, 2018

rerun tests

///printf("%s", palabra);
p = palabra;
///p = p+1;
///printf("%s\n", p);
Copy link
Owner

@gbobr gbobr Oct 12, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Acá debería ir printf("\n#SALIDA#\n");
Al faltar ese printf, no está pasando el test automatizado que se utiliza para corregir estos ejercicios. Por favor, agregue el print faltante.

Copy link
Owner

@gbobr gbobr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si bien, compila correctamente no está pasando el test automatizado. Por favor, revisar.

Por otro lado, debería entregar en un mismo pull request ambos ejercicios.

@gbobr
Copy link
Owner

gbobr commented Oct 12, 2018

run tests

@gbobr
Copy link
Owner

gbobr commented Oct 12, 2018

rerun tests

@gbobr
Copy link
Owner

gbobr commented Oct 12, 2018

rerun tests

@gbobr
Copy link
Owner

gbobr commented Oct 12, 2018

rerun tests

@gbobr
Copy link
Owner

gbobr commented Oct 18, 2018

add to whitelist

@gbobr
Copy link
Owner

gbobr commented Oct 18, 2018

Falla el test automatizado, aparentemente el ordenamiento no es correcto o está al revéz

Ver salida de la compilación en: https://jenkins-gbobr-ci.7e14.starter-us-west-2.openshiftapps.com/job/punteros2/34/console

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants