Nuestro proceso de desarrollo es el siguiente
codifique la tarea -> el código y la documentación de QA de otra persona -> la tarea se fusiona en el tronco.
Recientemente, un colega se niega a pasar el código de control de calidad debido a problemas con la sangría y los espacios en blanco.
Aquí hay ejemplos de estos problemas (la sintaxis es SAS):
Espacio en blanco adicional:
%if &syserr gt 0 %then %goto err; /*last line of code*/
/* Footer area*/
Línea adicional de espacio en blanco, y sin sangría dentro del proceso de clasificación:
/* End Of header * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
proc sort data = %dataset ;
by id;
run;
%if &syserr gt 0 %then %goto err;
proc sort data = &second_dataset.;
by id;
run;
%if &syserr gt 0 %then %goto err;
Espacio en blanco adicional entre pasos:
/*join all details on for each record*/
proc sort data = &data out = data_srt ;
by &conditions;
run;
%if &syserr gt 0 %then %goto err;
proc sort data = &data2.;
by &conditions.;
run;
%if &syserr gt 0 %then %goto err;
/*cartesian join*/
data new_data;
join data
&data2. ;
by &conditions;
run;
La pregunta es, al ser un buen programador, ¿está revisando su código y corrigiendo todo este tipo de cosas lo correcto, o es simplemente ridículo?
Existe una complicación adicional, que debido a que no tenemos una integración continua o pruebas automatizadas, no es posible que el QAer solucione rápidamente estos problemas y confirme el código, por el riesgo de eliminar el punto y coma accidentalmente o algo así. (Para ser justos, el riesgo se aplica al desarrollador inicial que realiza estos cambios, por lo que, de cualquier manera, si se produce este error, solo debe repararse y seguir adelante).