From e376b596a768421a502af5a7a85879962800ccc9 Mon Sep 17 00:00:00 2001 From: Salim Fadhley Date: Sun, 3 Aug 2014 00:16:13 +0100 Subject: [PATCH] Fix some pep8 violations I had introduced. --- jenkinsapi/jenkinsbase.py | 5 +++-- jenkinsapi/job.py | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/jenkinsapi/jenkinsbase.py b/jenkinsapi/jenkinsbase.py index 605bba8..ddd4d92 100644 --- a/jenkinsapi/jenkinsbase.py +++ b/jenkinsapi/jenkinsbase.py @@ -10,6 +10,7 @@ from jenkinsapi.custom_exceptions import JenkinsAPIException class JenkinsBase(object): + """ This appears to be the base object that all other jenkins objects are inherited from """ @@ -33,7 +34,8 @@ class JenkinsBase(object): self.poll() def get_jenkins_obj(self): - raise NotImplementedError('Please implement this method on %s' % self.__class__.__name__) + raise NotImplementedError( + 'Please implement this method on %s' % self.__class__.__name__) def __eq__(self, other): """ @@ -60,7 +62,6 @@ class JenkinsBase(object): def _poll(self): url = self.python_api_url(self.baseurl) return self.get_data(url) - def get_data(self, url, params=None): requester = self.get_jenkins_obj().requester diff --git a/jenkinsapi/job.py b/jenkinsapi/job.py index 8724216..c3b9e9f 100644 --- a/jenkinsapi/job.py +++ b/jenkinsapi/job.py @@ -200,7 +200,7 @@ class Job(JenkinsBase, MutableJenkinsThing): queue_url = response.headers['location'] qi = QueueItem(queue_url, self.jenkins) - + if block: qi.block_until_complete(delay=10) return qi -- 2.34.1