Maven SCM
  1. Maven SCM
  2. SCM-133

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
    • 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

        Hide
        Trygve Laugstøl 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 Laugstøl 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
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: