Merge redundant repository classes

Details

Type: Improvement
Status: Open
Priority: Major
Resolution: Unresolved
Affects Version/s: 1.0-beta-2
Fix Version/s: 2.0
Component/s: maven-scm-api
Labels:
None

Complexity:
Intermediate
Number of attachments :
0

Description

Currently there are:

com.apache.maven.scm.repository.ScmRepository
com.apache.maven.scm.provider.ScmProviderRepository

The comment at the top of the former says:

@todo clarify need - should be able to merge with ScmProviderRepository?

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Trygve Laugst�l added a comment - 16/Jan/06 4:26 PM

IIRC this was to minimize the info to the provider to exactly what it needed. There's a lot of code that's shared between and hidden from the providers and instead put in the API.

Show

Trygve Laugst�l added a comment - 16/Jan/06 4:26 PM IIRC this was to minimize the info to the provider to exactly what it needed. There's a lot of code that's shared between and hidden from the providers and instead put in the API.

Emmanuel Venisse made changes - 02/Feb/06 10:54 AM