Activity CacheAdminA_trunk2010/HADWIZ.mac.rou

4:55 pm

Wim Vermeulen

More
CR-2872
started review

[ICT-3847][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 : code productie-waardig maken...

1:37 pm

No big deal. Codebase in t Nederlands, maar StringUtils.Replace() is Engels, dus wellicht geen taal-reset van mijn brains http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/em...

No big deal.
Codebase in t Nederlands, maar StringUtils.Replace() is Engels, dus wellicht geen taal-reset van mijn brains

Maar in feite zetten we geen commentaar in onze code (= clean code principle!)
Hier, in dit specifiek geval is het echter om te benadrukken dat ALLE spaties moeten vervangen worden.
Uitzondering bevestigt de regel.

8:12 am

Matthias Claessens

More
CR-2604
finished reviewing

8:12 am

Matthias Claessens

More
Re: CR-2604
commented on review

Algemene vraag: Ik heb als PM tester enkele opmerkingen in het Jira ticket geplaatst. Moet ik hier al reviewer rekening mee houden tijdens de review? De uitleg van Tom was dat de reviewer enkel d...

Algemene vraag:

Ik heb als PM tester enkele opmerkingen in het Jira ticket geplaatst.
Moet ik hier al reviewer rekening mee houden tijdens de review?

De uitleg van Tom was dat de reviewer enkel de kwaliteit van de code moet beoordelen.
Maar er is een optie 'factually incorrect' in de tool, en mogelijks vallen enkele van mijn opmerkingen onder deze noemer?

Graag advies.

7:36 am