Hi list,

I have made a PR with some code cleanup and slight redesign for Medic, and I 
would like to please request a code review from folks in the community when it 
would be convenient for them! The PR is here: 
https://github.com/apache/cordova-medic/pull/37. For those interested in the 
details but not in reviewing the code, below are a list of changes and their 
justifications and a list of the specific incompatibilities with the previous 
medic code.

Code Changes and Reasons:
1.       Putting private data into a private file: this was done to remove 
passwords from the repo
2.       Splitting config into several files: this was done so that the config 
can be extended locally without affecting other installations that share code 
(i.e. Apache Infra Buildbot)
3.       Removing PlatformTestBase and its child classes: this was done in 
favour of using build properties to modify build steps instead of hard platform 
code as much as possible
4.       Using Buildbot Properties to pass parameters to builds: this was done 
so that medic doesn’t need to be reconfigured/restarted to run parametrized 
builds, which allows it to run in Apache Infrastructure and enables faster 
development
5.       Adding step wrappers: this was done to reduce the probability of 
errors by setting sensible defaults for values that are easily forgotten
6.       Removing categories from repo config: this was done to reduce code 
complexity for the checkout steps

Incompatibilities:
1.       The format of the `cordova-repos.json` file has changed; current files 
will need to simply be reformatted to match the new format, and they will work 
with the new medic code
2.       Semantics of the `cordova-config.json` file have changed, and current 
setups will stop being able to control some of medic’s behaviour using their 
current files; namely, builders and schedulers are now configured in 
cordova.conf, and  some platform settings have been removed (e.g. iOS keychain 
information)

Kindly,
Dmitry

Reply via email to