Closed thijsbrentjens closed 6 years ago
lvdbrink
commented
6 years ago Ik weet niet hoe mijn 'build by lvdb' in deze PR terecht gekomen is (misschien per ongeluk, ik heb dit niet bewust gedaan). Hoort dat hier wel thuis?
Er zit een conflict in package-lock.json, dat kan je denk ik oplossen.
Weet je waarom de Travis CI build gefaald is?
thijsbrentjens
commented
6 years ago De Travis build gaat mis op de W3C voorbeelden / tests. Willen we die behouden in onze code? Of willen we tests hebben voor (alleen) Geonovum respec documenten? @lvdbrink hoe gebruikten we die Travis builds eerst?
lvdbrink
commented
6 years ago Wat mij betreft is het de bedoeling om tests te hebben voor Geonovum respec documenten. Die hebben we denk ik nog niet? Het was als ik me goed herinner iets waar we nog een keer aan moesten werken. Travis hebben we ooit toegevoegd met het idee om geonovum specifieke tests te maken, maar dat is er nog niet van gekomen.
Zie ook https://github.com/Geonovum/respec/issues/114
Van: Thijs Brentjens [notifications@github.com] Verzonden: maandag 4 juni 2018 9:51 To: Geonovum/respec Cc: Linda van den Brink; Mention Onderwerp: Re: [Geonovum/respec] Tb update mei18 (#137)
De Travis build gaat mis op de W3C voorbeelden / tests. Willen we die behouden in onze code? Of willen we tests hebben voor (alleen) Geonovum respec documenten? @lvdbrinkhttps://github.com/lvdbrink hoe gebruikten we die Travis builds eerst?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/Geonovum/respec/pull/137#issuecomment-394265311, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHk1W2DGfwyLSEwXgKtHpGUJsKkjqyf5ks5t5OcXgaJpZM4TynpL.
Update van veel (upstream) wijzigingen van W3C, inclusief aanpassingen aan header Geonovum. Nu via Javascript gegenereerd, in plaats van de HTML templates van eerder.