Review of architecture and solution

Praqma / pretested-integration-plugin

A generic pretest commit plugin for Jenkins CI

MIT License

7 stars 14 forks source link

Closed buep closed 7 years ago

buep commented 7 years ago

@bicschneider proposes @MadsNielsen should review the solution in general.

I that case I want a list of suggested improvements as output, including reason on why to do them and their importance/impact in not done.

buep commented 7 years ago

Estimate is half a day for two persons = 1 day.

buep commented 7 years ago

I have now also been over the code myself, and many things found and changed after Claus changes so we're in an okay state now with recent changes.

Will thus close this issue.