Organize interface

Abierto
#920 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
15/100
Tipo de issue
Refactorización
Claridad
Necesita aclaración
Estado de actividad
Activo
Stack tecnológico
cpp
Área
backend

Línea de trabajo

The issue names no files, tests or entry points, so start by asking the maintainers which dependency-injected interfaces are in scope and what the expected pattern is. Survey where raw pointers, smart pointers and references are passed as constructor or function parameters for mocking. Done means the maintainers have agreed on a convention and the listed interfaces follow it, with the existing tests still passing.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

What

We have a lot of interfaces and raw pointers / smart pointers / references to the implementation passed into parameters for dependency injection mocking.
Ideally we would remove all raw pointers for smart pointers, then if it's impossible for the object to be nullptr then we use references, however these are not default constructable only with a std::ref.

Acceptance Criteria (DoD)
  • All dependency injected interfaces aligned to the expectations.
How

No response

Lenguaje dominante
C++
Estrellas
7
Forks
37
Merge medio
2 d 11 h
PR fusionados (30 d)
106

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

  • Sin Dockerfile ni archivo de Docker Compose
  • Tiene una plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de eclipse-score/lifecycle

Todos los issues de eclipse-score/lifecycle

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.