Closed jspijker closed 2 years ago
De checks zijn gefaald, maar volgens mij alleen met warnings. @jspijker is dit een probleem volgens jou?
De checks zijn gefaald, maar volgens mij alleen met warnings. @jspijker is dit een probleem volgens jou?
@florisnaus De checks falen op warnings en errors, alleen de 'notes' worden geaccepteerd. Dit is hetzelfde beleid als bij CRAN. Die warnings geven toch meestel serieuze kwaliteits-issues aan (in ons geval documentatie die achter loopt met de implementatie).
@jspijker Duidelijk. Ik zag in de docs staan dat deze niet handmatig maar automatisch aangevuld moeten worden. Moet die automatische handeling wel handmatig worden opgestart, of gaat er iets mis daarmee?
@jspijker Nu zijn de checks wel goed gegaan. Wil je nog dat we alle changes opnieuw gaan reviewen, of kan het nu gewoon naar master worden gemerged?
Deze PR wordt uiteindelijk een volgende release.