`TestsShouldNotBePublic` should reduce visibility if used elsewhere in Javadoc links
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 43/100
Línea de trabajo
Comienza leyendo la receta TestsShouldNotBePublic y el fallo informado en src/test/java/org/jabref/logic/bst/BstFunctionsTest.java, incluida su importación de BstCaseChangersTest. Comprueba cómo interactúan los cambios de visibilidad con las referencias en otras partes y los enlaces de Javadoc. Se considera terminado cuando aplicar la receta ya no introduce el error de compilación informado y la reducción de visibilidad prevista sigue produciéndose.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I applied TestsShouldNotBePublic to JabRef's code.
I get compile errors afterwards:
/home/runner/work/jabref/jabref/src/test/java/org/jabref/logic/bst/BstFunctionsTest.java:7: error: BstCaseChangersTest is not public in org.jabref.logic.bst.util; cannot be accessed from outside package
import org.jabref.logic.bst.util.BstCaseChangersTest;
I know, this is bad software design. However, I think, rewriting should not introduce compile errors.
- Lenguaje dominante
- Java
- Estrellas
- 100
- Forks
- 107
- Merge medio
- 3 h 47 min
- PR fusionados (30 d)
- 71
Preparar el entorno
- Sin Dockerfile ni 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 openrewrite/rewrite-testing-frameworks
-
good first issue
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
openrewrite/rewrite-testing-frameworks#1211 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
JUnit 4 to 5 Migration: Tests with Multiple @Before Methods Fail Due to Arbitrary Execution OrderPosiblemente ocupada @ApekshaAmbala la tomó hace 2 días. Abiertojunit recipe
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
openrewrite/rewrite-testing-frameworks#1207 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
JUnit4to5Migration - undiscovered or invalid Kotlin testsPosiblemente ocupada @hugoncosta la tomó hace 11 días. Abiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
openrewrite/rewrite-testing-frameworks#1188 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
openrewrite/rewrite-testing-frameworks#1167 ·
Los mantenedores suelen responder en 1 día
-
`ParameterizedRunnerToParameterized` silently skips classes whose `@Parameters` method is inherited, leaving a partially migrated class thatPosiblemente ocupada @mohitduhan19 la tomó hace 25 días. Abiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 68/100
openrewrite/rewrite-testing-frameworks#1126 ·
Los mantenedores suelen responder en 1 día
Todos los issues de openrewrite/rewrite-testing-frameworks
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
AllTheMods/AllTheModium#238 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
apache/stormcrawler#2264 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
commons-app/apps-android-commons#7115 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
objectionary/lints#1596 ·
Los mantenedores suelen responder en 2 días