Activity CacheAdminA_trunk2010/vhUnitTest/End2End/TAOR

11:06 am

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken. Alvast bedankt om het aantal testfiles te filteren. Het geheel lijkt me we...

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken.
Alvast bedankt om het aantal testfiles te filteren.
Het geheel lijkt me wel oké te zijn.

Ook in eCon enkele commits gereviewed.

6:38 pm

Wim Vermeulen

More
Re: CR-2513
commented on review

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten) In combinatie met svn rev. 69413 is dit wel een correcte overgang. Hierbij is deze cha...

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten)
In combinatie met svn rev. 69413 is dit wel een correcte overgang.

Hierbij is deze change nagekeken en goedgekeurd!

10:35 am

Tommy Hebb

More
Re: CR-2513
commented on review

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om ui...

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om uit te maken a.d.h.v. de changesets of het al dan niet over code gaat of testfiles.
P.s.: Ook kleine change in Econ (zie svn).

10:31 am

Tommy Hebb

More
CR-2513
started review

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

1:58 pm

Jo Willems

More
CR-1891
Closed

8:07 am

Tom Vermeulen

More
CR-1891
finished reviewing

8:07 am

Tom Vermeulen

More
Re: CR-1891
commented on review

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronde...

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronder verifiëren?

8:01 am

Tom Vermeulen

More
Re: CR-1891
commented on review

overbodige comment?

2:35 pm

Jo Willems

More
CR-1891
started review

[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten

10:08 am

Line History

Stats

Commits this week: 0

Commit Activity

52 week commits volume

Commits by day

Commits by hour

Commit calendar

Most active committers (90 days)