luckyframework/avram

Operation callbacks are missing the conditional arguments

Ouverte

#1 066 ouverte le 25 sept. 2024

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Crystal (67 forks)auto 404
hacktoberfest

Métriques du dépôt

Stars
 (180 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

https://github.com/luckyframework/avram/blob/bb63db88513b05e80030a4c61c9ef48b0ea12dbe/src/avram/callbacks.cr#L97C9-L97C19

With SaveOperation and DeleteOperation you can do before_save :do_thing, if: :valid?, but this doesn't work for before_run or after_run

Guide contributeur

Direction de recherche
Examinez l'implémentation des rappels dans Avram pour comprendre comment les arguments conditionnels sont passés aux rappels before save et before run/after run. Consultez le code à la ligne de référence et tracez la manière dont l'option if: est traitée pour les différents types de rappels.
Stack technique
Aucun
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65