From 8dc46c16ceefa1c70108105674a963486b688c7e Mon Sep 17 00:00:00 2001 From: Pablo Hoffman Date: Sun, 5 Oct 2008 07:35:58 +0000 Subject: [PATCH] removed old comment --HG-- extra : convert_revision : svn%3Ab85faa78-f9eb-468e-a121-7cced6da292c%40299 --- scrapy/trunk/scrapy/spider/models.py | 6 ------ 1 file changed, 6 deletions(-) diff --git a/scrapy/trunk/scrapy/spider/models.py b/scrapy/trunk/scrapy/spider/models.py index 48125133a..d680b93e9 100644 --- a/scrapy/trunk/scrapy/spider/models.py +++ b/scrapy/trunk/scrapy/spider/models.py @@ -8,12 +8,6 @@ from scrapy.core.exceptions import UsageError def _valid_start_urls(obj): """Check the start urls specified are valid""" - # TODO: we should add proper validation here - - ## This commented if is a test i was doing, it also takes in account the spider's base class (if any) - ## it works, but the attribute is checked somewhere else (zope?) and causes to fail - ##if not (obj.start_urls or any([hasattr(base, 'start_urls') for base in list(obj.__class__.__bases__)])): - if not obj.start_urls: raise UsageError("A start url is required")