search

clessn / hublotr

Package R d'accès à clHub
Other
3 stars 0 forks source link

Add MIT license #43

Closed judith-bourque closed 1 year ago

judith-bourque commented 1 year ago

@olichose123 Ça fonctionne si ce package est assigné la license MIT?

Ça va permettre d'automatiser les tests à l'aide de GitHub Actions. clessnverse utilise déjà la license MIT.

olichose123 commented 1 year ago

Très bonne idée!

olichose123 commented 1 year ago

Par contre c'est pas supposé avoir d'impact sur les workflows?

judith-bourque commented 1 year ago

Par contre c'est pas supposé avoir d'impact sur les workflows?

Quand on ajoute un workflow pour les tests automatisés, ils roulent sans problème. Cependant, quand le package n'a pas de license, ce workflow va toujours s'arrêter pour dire d'ajouter une license (exemple).

Avant d'ajouter le workflow sur master, je voulais adresser l'erreur puisque les messages sont plus difficiles à ignorer dans GitHub Actions que dans ma console R 😅

olichose123 commented 1 year ago

Ah c'est ben bizz! Je croyais que c'était un requirement github mais non c'est juste R qui se plaint!

judith-bourque commented 1 year ago

C'est ça, c'est juste le workflow en spécifique qui le demande puisqu'il fait exactement la même chose que devtools::check().

Maintenant que le message d'erreur de license est réglé, on obtient maintenant de nouveaux messages d'erreurs à adresser éventuellement. Ils sont signalés dans #46.