diff options
author | Lars Wirzenius <lars.wirzenius@codethink.co.uk> | 2014-07-30 16:58:34 +0000 |
---|---|---|
committer | Lars Wirzenius <lars.wirzenius@codethink.co.uk> | 2014-07-30 17:20:18 +0000 |
commit | bedec8e16e987f7f4acb89ae7def5a702e1fcc9b (patch) | |
tree | 46d3b0859143f02f09bfeb6a0c5a1d4636d48030 /scripts | |
parent | 389b259fb586454c3dc447613545c280d512fc51 (diff) | |
download | definitions-bedec8e16e987f7f4acb89ae7def5a702e1fcc9b.tar.gz |
Simplify logic and condition
Avoiding a condition that has a negation tends to be a bit simpler
for humans to understand.
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/release-build | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/scripts/release-build b/scripts/release-build index 36ecb423..a3e25ebb 100755 --- a/scripts/release-build +++ b/scripts/release-build @@ -121,9 +121,8 @@ class ReleaseApp(cliapp.Application): builds = [] for system_name in systems: system, _ = self.load_morphology(system_name) - if system['arch'] not in self.controllers: - continue - builds.append(Build(system_name, system['arch'], self)) + if system['arch'] in self.controllers: + builds.append(Build(system_name, system['arch'], self)) return builds def deploy_images(self, cluster, cluster_path): |