diff --git a/jenkins_jobs/config.py b/jenkins_jobs/config.py index 031bb922d..9a6501019 100644 --- a/jenkins_jobs/config.py +++ b/jenkins_jobs/config.py @@ -265,9 +265,9 @@ class JJBConfig(object): plugins_info = None if (config.has_option(self._section, 'query_plugins_info') and - not config.getboolean(self._section, "query_plugins_info")): - logger.debug("Skipping plugin info retrieval") - plugins_info = [] + not config.getboolean(self._section, "query_plugins_info")): + logger.debug("Skipping plugin info retrieval") + plugins_info = [] self.builder['plugins_info'] = plugins_info self.recursive = config.getboolean('job_builder', 'recursive') diff --git a/jenkins_jobs/modules/publishers.py b/jenkins_jobs/modules/publishers.py index 126077cfc..38cf575c9 100644 --- a/jenkins_jobs/modules/publishers.py +++ b/jenkins_jobs/modules/publishers.py @@ -6265,7 +6265,7 @@ def conditional_publisher(registry, xml_parent, data): # Check the length of actions list for versions prior to 0.13. # Flexible Publish will overwrite action if more than one is # specified. Limit the action list to one element. - if len(actions) is not 1: + if len(actions) != 1: raise JenkinsJobsException("Only one action may be " "specified for each condition.") for action in actions: