search

rtcTo / rtc2git

A tool made for migrating code from an existing IBM's RTC SCM repository into a Git repository
https://rtc.to
MIT License
81 stars 60 forks source link

Svn repo #129

Closed veon82 closed 7 years ago

veon82 commented 7 years ago
WtfJoke commented 7 years ago

Hi Veon82

Thanks for the pull-request. Can you split the pull requests in at least three parts (1 for adjustments for scm Tools 6.0.4 and 1 for svn part and one for component2load).

Also check that the tests/build still work.

Then I will review the pull request more detailed.

Another question: I cant see any advantage for the newly introduced component2load. Can you provide more details/reason why you added this?

veon82 commented 7 years ago

Hi,

Can you split the pull requests in at least three parts (1 for adjustments for scm Tools 6.0.4 and 1 for svn part and one for component2load).

Yes, of course, I'll do it asap.

I cant see any advantage for the newly introduced component2load. Can you provide more details/reason why you added this?

The reason why I introduced this option is because I had to convert a single component of a workspace in a git repo. In my RTC repository I had a lot of components and I only needed to export a few ones. Maybe rtc2git could already do this and I didn't catch this functionality?

2017-11-01 15:17 GMT+01:00 Manuel notifications@github.com:

Hi Veon82

Thanks for the pull-request. Can you split the pull requests in at least three parts (1 for adjustments for scm Tools 6.0.4 and 1 for svn part and one for component2load).

Also check that the tests still work.

Then I will review the pull request more detailed.

Another question: I cant see any advantage for the newly introduced component2load. Can you provide more details/reason why you added this?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rtcTo/rtc2git/pull/129#issuecomment-341118824, or mute the thread https://github.com/notifications/unsubscribe-auth/ALvEm2vHIXtaM5PvGkpwnVXyVeN41DPBks5syH2DgaJpZM4QNDBW .

-- Andrea Baldini

Via Pesaro 9360126 Ancona