Closed ElsLommelen closed 3 years ago
bij deze heb ik dus de problemen i.v.m. het dynamisch validatierapport van issue #32 opgelost
Bij deze is ook het issue over functie initiatie (#34) opgelost
Hierbij zijn stappen gezet voor de oplossing van issues #36 en #41, zijn deze stappen voldoende? (Verder te bediscussiëren bij de issues in kwestie.)
@leymanan Hiermee heb ik de issues opgelost die jij aangebracht had. Terwijl je de oplossingen nakijkt en uittest (geen haast bij, liever dat je het grondig doet), zal ik issue #30 nog in orde brengen in deze PR. Daarna zal ik een review request sturen (maar geef gerust verder in de issues aan als iets nog niet volledig in orde is, dat werkt wel handig).
En dan stel ik voor om nadien in een aparte PR nog eventjes de afwerking van het package te doen (website met documentatie, versienummer en alvast 'news'-pagina aanmaken om bij nieuwe versies de aanpassingen toe te voegen, dhcurve toevoegen aan deze website,...).
Ik beschouw dit als afgewerkt, ben ik iets vergeten? (Issue #30 vind ik minder dringend omdat het geen impact heeft op de gebruikers, ik stel voor om dit uit te stellen voor een volgende versie.)
Is ook volgens mij afgewerkt! Bedankt! Ga jij mergen?
Grts, anja
Op di 15 dec. 2020 om 17:20 schreef ElsLommelen notifications@github.com:
Ik beschouw dit als afgewerkt, ben ik iets vergeten? (Issue #30 https://github.com/inbo/dhcurve/issues/30 vind ik minder dringend omdat het geen impact heeft op de gebruikers, ik stel voor om dit uit te stellen voor een volgende versie.)
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/inbo/dhcurve/pull/42#issuecomment-745399780, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGKXN5HBUVUCXWPFRGP3OZLSU6EEJANCNFSM4TFMFUJQ .
--
Anja Leyman
Expert Cel Beheerplanning en Monitoring
Ik werk tijdelijk niet op woensdag- en vrijdagnamiddag.
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
Vlaamse overheid
AGENTSCHAP NATUUR & BOS
Standplaats Instituut voor Natuur- en Bosonderzoek (INBO) Gaverstraat 4, 9500 Geraardsbergen T: 054 436 182 M: 0495 14 90 60 E-mail: anja.leyman@vlaanderen.be anja.leyman@lne.vlaanderen.be
De inhoud van dit bericht en eventuele bijlage(n) verbinden het Agentschap voor Natuur en Bos niet, zolang niet bevestigd door een geldig ondertekend document
Ok, ik zal mergen, en dan meteen de andere PR verder in orde brengen. Bedankt voor je hulp!
jij bedankt!!
Op wo 16 dec. 2020 om 10:12 schreef ElsLommelen notifications@github.com:
Ok, ik zal mergen, en dan meteen de andere PR verder in orde brengen. Bedankt voor je hulp!
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/inbo/dhcurve/pull/42#issuecomment-745963272, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGKXN5G6XX3YKI6EFPPKVT3SVB2ZFANCNFSM4TFMFUJQ .
--
Anja Leyman
Expert Cel Beheerplanning en Monitoring
Ik werk tijdelijk niet op woensdag- en vrijdagnamiddag.
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
Vlaamse overheid
AGENTSCHAP NATUUR & BOS
Standplaats Instituut voor Natuur- en Bosonderzoek (INBO) Gaverstraat 4, 9500 Geraardsbergen T: 054 436 182 M: 0495 14 90 60 E-mail: anja.leyman@vlaanderen.be anja.leyman@lne.vlaanderen.be
De inhoud van dit bericht en eventuele bijlage(n) verbinden het Agentschap voor Natuur en Bos niet, zolang niet bevestigd door een geldig ondertekend document
Aanpassing van de verouderde code aan recente aanpassingen in packages (bv. nieuwe semantiek voor dplyr) voor zover dit nodig is voor de goede werking van het package.