Java library should not use singletons
@joakim-olsson ya está trabajando en esto.
Desde el 1/3/2022.
- #675 de @SimonOsnes — cerrado sin fusionar
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- java
- Área
- api, backend-api-design
Línea de trabajo
Start by reviewing the current global initialization around Twilio.init and the context from followup issue #430. Compare the existing client structure with the proposed TwilioApi interface and Twilio implementation. Done means establishing whether this architectural change is feasible for version 9 while preserving the stated consumer use cases.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
As a followup to #430, while using global setup such as Twilio.init is ordinary practice in some languages, it is contrary to Java standard practices and substantially interferes in standard Java application architectures.
For version 9 development, your customers would be much better served by following the standard layout of a service interface TwilioApi and a class Twilio implements TwilioApi; this would not impose any more significant burden on consumers (just new Twilio(username, password) instead of Twilio.init(username, password)) but would dramatically enhance testability and some production use cases such as multitenant implementations.
- Lenguaje dominante
- Java
- Estrellas
- 529
- Forks
- 452
- Merge medio
- 1 h 30 min
- PR fusionados (30 d)
- 3
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una 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 twilio/twilio-java
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
twilio/twilio-java#976 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
twilio/twilio-java#975 ·
-
[BUG] Sip.Builder(String) drops SIP destinations with an explicit port when the scheme is omittedAbiertotype: bug
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
twilio/twilio-java#982 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
twilio/twilio-java#960 ·
-
MessageIntent is not supported in the Java SDKQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
twilio/twilio-java#950 · 2 comentarios ·
Todos los issues de twilio/twilio-java
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
apache/rocketmq-dashboard#7250 ·
Los mantenedores suelen responder en 3 días
-
HttpMaxFrameSizeTest.sendOversizedMessageTest() passes silently when maxFrameSize is not enforcedAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
apache/activemq#2706 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
federicoiosue/Omni-Notes#1032 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
unitycatalog/unitycatalog#2025 ·
Los mantenedores suelen responder en 4 días
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
krizaka/krizaka-notifications#22 ·
Los mantenedores suelen responder en 1 día