Merge redundant repository classes

Details

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

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

Hide
Permalink
Trygve Laugstol added a comment -

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 Laugstol added a comment - 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.

People

  • Assignee:
    Unassigned
    Reporter:
    Mike Perham
Vote (0)
Watch (0)

Dates

  • Created:
    Updated: