Summary

Remove legacy resource specific code.

Review Request #2847 — Created Feb. 9, 2012 and discarded May 10, 2012, 3:36 a.m.

Information

Owner

mbait

Repository

RBTools

Branch

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

The new implementation uses resource generator so the most of legacy classes are useless. This patch also removes the most of errors defined in rbtools.api.errors.

Testing Done

Summary:

Fix static erros and notifications in resource.py

Remove legacy resource specific code.

Description:

~		Cleanup pep8 and statick checker notifications in order to simplify further reviews. The code structure is still broken.
	~	The new implementation uses resource generator so the most of legacy classes are useless. This patch also removes the most of errors defined in rbtools.api.errors.

Diff:

Revision 2 (+8 -778)

Show changes

	rbtools/api/errors.py
	rbtools/api/resource.py

The generator isn't here yet, right? Might be best to have that in first.

I know we have that thread, but given the scope of this project, I think it would be very helpful for us when reviewing and discussing if we had a sort of living spec on this. The reason being that this set of changes is very crucial to the future of RBTools, and I want to be sure everyone is on the same page before any commits go in (since these are all small commits as part of your work, but we have no whole to see right now). Would you be able to write up a sort of spec on the wiki of how this will all work?

Status:: Discarded