getsentry / getsentry/sentry-ruby

Cleanup initialize/close behavior to prevent initialized? checks everywhere

Abierto
#2,096 1 comentario 0 reacciones 1 asignado Ver en GitHub

Nadie ha tomado este issue todavía.

Refactoring Ruby Task
Lenguaje dominante
Ruby
Estrellas
988
Forks
542
Merge medio
17 h 40 min
PR fusionados (30 d)
19

Descripción

And perhaps we should implement a better version of close and Sentry.initialize? 🤔

Specifically:

  • We don't erase hub variables, @background_worker and @session_flusher
  • Sentry.initialize? is backed by a simple boolean state that's changed by Sentry.ini and Sentry.close.

We would still need Sentry.initialize? in some places to stop some actions after the SDK is closed, but we don't need it for preventingnil value anymore, which is getting out of hand IMO.

Originally posted by @st0012 in https://github.com/getsentry/sentry-ruby/pull/2084#discussion_r1308034537

Guía de contribución

Abrir la 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.

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.