Maven SCM
  1. Maven SCM
  2. SCM-630

Request to include MKS Integrity SCM Provider along with the rest of the Maven SCM Providers

    Details

    • Type: New Feature New Feature
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.6
    • Component/s: maven-scm-api
    • Labels:
      None
    • Environment:
      Unix and Windows
    • Complexity:
      Intermediate
    • Patch Submitted:
      Yes
    • Number of attachments :
      7

      Description

      I've developed an SCM Provider for MKS Integrity.
      I would like to contribute this code to the Apache community for inclusion on the Maven SCM Provider list

        Activity

        Hide
        Cletus D'Souza added a comment -

        Sure, I can work on creating the unit tests. Any idea when maven scm 1.6 is scheduled to be released?

        Show
        Cletus D'Souza added a comment - Sure, I can work on creating the unit tests. Any idea when maven scm 1.6 is scheduled to be released?
        Hide
        Olivier Lamy added a comment -

        No idea yet for the release.
        I have to review some issues first.

        Show
        Olivier Lamy added a comment - No idea yet for the release. I have to review some issues first.
        Hide
        Cletus D'Souza added a comment - - edited

        Attached are the unit tests that make the most sense. Some tests are intentionally excluded as some test commands execute more than one scm command (ex: login - consumed by all commands, edit - consumed by checkin, etc.). Also, I've excluded tests for CLI based commands (blame, fileinfo, and diff) as it doesn't make sense to simply assert the command line string. It almost seems like hardcoding it in two places, so theres little value. Hopefully these tests are sufficient for the intial publication/release.

        Show
        Cletus D'Souza added a comment - - edited Attached are the unit tests that make the most sense. Some tests are intentionally excluded as some test commands execute more than one scm command (ex: login - consumed by all commands, edit - consumed by checkin, etc.). Also, I've excluded tests for CLI based commands (blame, fileinfo, and diff) as it doesn't make sense to simply assert the command line string. It almost seems like hardcoding it in two places, so theres little value. Hopefully these tests are sufficient for the intial publication/release.
        Hide
        Olivier Lamy added a comment -

        Thanks !
        I have excluded some by default. You can run it tru the profile tck-mks.

        Show
        Olivier Lamy added a comment - Thanks ! I have excluded some by default. You can run it tru the profile tck-mks.
        Hide
        Olivier Lamy added a comment -

        we are fine IMHO

        Show
        Olivier Lamy added a comment - we are fine IMHO

          People

          • Assignee:
            Olivier Lamy
            Reporter:
            Cletus D'Souza
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: