Code review will not fail automatically in case of issues
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Error
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- github-actions, javascript
Línea de trabajo
Empieza por rastrear el punto de entrada de la acción de revisión de código responsable de gestionar el resultado de revisión mostrado y el timeout de seis horas. Reproduce o inspecciona la ruta del fallo informada y, a continuación, verifica que una revisión fallida provoque que la acción falle o se cancele de inmediato, en lugar de esperar al timeout.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción

For some reason, there was an issue with the review
and action didn't cancel until the 6-hour timeout had elapsed
@@ -6,6 +6,7 @@
import lombok.Getter;
import java.math.BigDecimal;
import java.math.RoundingMode;
@Getter
@Builder(access = AccessLevel.PRIVATE)
@@ -21,6 +22,8 @@ public class SubscriptionPlanDto {
public static SubscriptionPlanDto NONE() {
// TODO: 구독 없는 경우 어떻게 처리? (플로우)
return SubscriptionPlanDto.builder()
.planCode("")
.price(BigDecimal.ZERO)
.name("이용중인 서비스 없음")
.isActive(false)
.build();
@@ -34,7 +37,7 @@ public static SubscriptionPlanDto of(SubscriptionPlan plan) {
.description(plan.getDescription())
.planCode(plan.getPlanCode())
.price(plan.getPrice())
.priceIncludeVat(plan.getPrice().multiply(BigDecimal.valueOf(1.1)))
.priceIncludeVat(plan.getPrice().multiply(BigDecimal.valueOf(1.1)).setScale(0, RoundingMode.FLOOR)) // 1원단위 절삭
.isActive(plan.getIsActive())
.build();
}
This code that tried, just simple
- Lenguaje dominante
- JavaScript
- Estrellas
- 4.5k
- Forks
- 466
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de anc95/ChatGPT-CodeReview
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
anc95/ChatGPT-CodeReview#76 · 2 comentarios ·
-
Include/ignore patterns are matched against the GitHub API URL instead of the repository-relative pathPosiblemente ocupada @strawberry-little-bear la tomó hace 6 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 25/100
anc95/ChatGPT-CodeReview#215 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
anc95/ChatGPT-CodeReview#208 ·
-
在action执行过程中有报错,action已完成但没有评论Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
anc95/ChatGPT-CodeReview#203 · 2 comentarios · 1 reacción ·
-
Rate limitPosiblemente ocupada @Br1an67 la tomó hace 241 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
anc95/ChatGPT-CodeReview#201 · 4 comentarios ·
Todos los issues de anc95/ChatGPT-CodeReview
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
micromatch/picomatch#243 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
b310-digital/teammapper#1515 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
c1overzz/dsh-bili-scroll#153 ·
Los mantenedores suelen responder en 1 día
-
01 type: bug 21 status: confirmed 41 lvl: easy 62 scope: viewer
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día