From db8839351479c60234bb6873394d93b56f0174eb Mon Sep 17 00:00:00 2001 From: "Bryn M. Reeves" Date: Wed, 12 Mar 2014 19:36:02 +0000 Subject: [PATCH 33/72] Rename validatePlugin to validate_plugin This one somehow didn't get the memo about camelCase not being cool any more in sos. Rename the method to comply with pep8 and not be so ugly. Signed-off-by: Bryn M. Reeves --- sos/policies/__init__.py | 2 +- sos/sosreport.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/sos/policies/__init__.py b/sos/policies/__init__.py index c272e5c..b91d0fd 100644 --- a/sos/policies/__init__.py +++ b/sos/policies/__init__.py @@ -198,7 +198,7 @@ No changes will be made to system configuration. return tempfile.gettempdir() return opt_tmp_dir - def validatePlugin(self, plugin_class): + def validate_plugin(self, plugin_class): """ Verifies that the plugin_class should execute under this policy """ diff --git a/sos/sosreport.py b/sos/sosreport.py index 0faa364..fe78abd 100644 --- a/sos/sosreport.py +++ b/sos/sosreport.py @@ -763,7 +763,7 @@ class SoSReport(object): tuple(self.policy.valid_subclasses)) for plugin_class in plugin_classes: - if not self.policy.validatePlugin(plugin_class): + if not self.policy.validate_plugin(plugin_class): self.soslog.warning(_("plugin %s does not validate, skipping") % plug) if self.opts.verbosity > 0: self._skip(plugin_class, _("does not validate")) -- 1.9.3