Romain Quetiez

> @rquetiez on \Combodo\iTop\Test\UnitTest\Core\ModelFactoryTest::testAlterationsByAPIs I've got an error on the `@covers \MFElement::GetDelta` phpdoc : > > > [EA] @Covers referencing to a non-existing entity '\MFElement::GetDelta' > > I don't see...

Tests are missing. We've prototyped some tests in BulkChangeTest and it proved that the currrent implementation does handle only specific cases. The following concerns have to be discussed and decided...

Reviewed today with Anne-Cath: 1. Missing new test cases (see request-template) 2. The implementation in MakeRealValue implies a change in the expected behavior (at least for the REST API) :...

Have you investigated means to ignore errors by the mean of PHP anotations? Phan allows that : https://github.com/phan/phan/wiki/Annotating-Your-Source-Code#phan-suppress-next-line Though it would be cumbersome to establish the first baseline, then the...

Data migration during might have an impact on the duration of the setup. This impact has to be evaluated and documented.