Skip to content

Conversation

aollier
Copy link
Contributor

@aollier aollier commented Oct 23, 2017

I made varied corrections:

  • moved a comma at its right place
  • replaced "working repository" by "working directory"
  • reworded the sentence about fetching the commits from the SVN repository and commiting them in the local Git repository
  • emphasised "dcommit"

- put the comma at the right place
- use "working directory" instead of "working repository"
It is supposed to be a personal SVN test repository
because the original one is no more available,
so we don't know how many commits there are.
(((Git as a client)))
Git provides such a nice experience for developers that many people have figured out how to use it on their workstation, even if the rest of their team is using an entirely different VCS.
There are a number of these adapters, called ``bridges,'' available.
There are a number of these adapters, called ``bridges'', available.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had a feeling about this, and after some research it seems I was correct: the comma goes inside the quotation marks.

The Punctuation Guide:

Commas and periods that are part of the overall sentence go inside the quotation marks, even though they aren’t part of the original quotation.

GrammarBook.com:

Rule 4. Periods and commas ALWAYS go inside quotation marks.

This runs the equivalent of two commands – `git svn init` followed by `git svn fetch` – on the URL you provide.
This can take a while.
The test project has only about 75 commits and the codebase isn't that big, but Git has to check out each version, one at a time, and commit it individually.
If, for example, the test project has only about 75 commits and the codebase isn't that big, Git has nevertheless to check out each version, one at a time, and commit it individually.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we change this to "Git nevertheless must check out…"?

@ben ben merged commit 516c491 into progit:master Oct 23, 2017
@aollier aollier deleted the git_svn branch October 24, 2017 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants