Quartznet: Envisagez de rendre publiques les nouvelles propriétés internes de QuartzOptions
Salut @lahma ,
Serait-il possible de rendre publiques les 3 propriétés actuellement internes ? Cela aiderait beaucoup avec les tests unitaires (pour s'assurer que les détails du travail et du déclencheur sont correctement enregistrés par une classe IConfigureOptions<QuartzOptions> ).
Merci!
kevinchalet
Tous les 7 commentaires
Il existe maintenant des getters publics en lecture seule pour les déclencheurs et les tâches, j'ai supprimé le calendrier pour le moment car il n'était même pas utilisé.
lahma
le 2 oct. 2020
Merci!
kevinchalet
le 2 oct. 2020
Wooops, je soupçonne que vous avez oublié de rendre les propriétés "publiques": https://github.com/quartznet/quartznet/commit/94a4f33968a259f7f2d9f37e62f79ebad58455da#diff -baab1cc10703ed0030c47a179783803aR36 😅
kevinchalet
le 2 oct. 2020
Oh cher...
lahma
le 2 oct. 2020
Maintenant, ça devrait être en maître, cette fois pour de vrai... 🤦
lahma
le 3 oct. 2020
Hey @kevinchalet Je viens de publier 3.2.1 sur NuGet et cette fois j'espère que le modificateur de visibilité est correct 😉
lahma
le 18 oct. 2020
@lahma génial, merci ! Je peux confirmer qu'ils sont maintenant vraiment "publics"
https://github.com/openiddict/openiddict-core/pull/1137
kevinchalet
le 18 oct. 2020
Questions connexes
Pinguwien
·
6Commentaires
frapid
·
45Commentaires
DixonDs
·
27Commentaires
dbraaten42
·
4Commentaires
ljani
·
9Commentaires