Unify SSH and HTTP clone & commit articles (#129) #143

Merged
fnetX merged 5 commits from issue-129 into master 11 months ago
n commented 11 months ago
Collaborator

This PR unifies the SSH and HTTP clone & commit articles into one Clone & Commit via CLI article and expands it a little.

TODO: Redirect the old pages. Unfortunately in Eleventy, setting an alias is not as straightforward compared to a SSG like Hugo.

This PR unifies the *SSH and HTTP clone & commit* articles into one *Clone & Commit via CLI* article and expands it a little. TODO: Redirect the old pages. Unfortunately in Eleventy, setting an alias is not as straightforward compared to a SSG like Hugo.
n added the
Status: In progress
Kind: Documentation
Kind: Enhancement
labels 11 months ago
n added 2 commits 11 months ago
Collaborator

lgtm

For the sake of completeness an example for a git pull output would be nice, we already have such examples on every other command listed in the document.

I think an "additional ressources" section (as mentioned in #135) with links to the official Git docs would work well to suplement our basic instructions.

lgtm For the sake of completeness an example for a `git pull` output would be nice, we already have such examples on every other command listed in the document. I think an "additional ressources" section (as mentioned in #135) with links to the official Git docs would work well to suplement our basic instructions.
n added 1 commit 11 months ago
n added 1 commit 11 months ago
n commented 11 months ago
Poster
Collaborator

@rwa sounds good for another PR, but I don't want to add anything else here :)

@rwa sounds good for another PR, but I don't want to add anything else here :)
n added
Status: Ready for Review
and removed
Status: In progress
labels 11 months ago
n commented 11 months ago
Poster
Collaborator

I added a redirect mechanism in 79463b300e so this should be ready for review and merge.

I added a redirect mechanism in 79463b300e so this should be ready for review and merge.
fnetX approved these changes 11 months ago
fnetX left a comment
Collaborator

lgtm

lgtm
rwa added 1 commit 11 months ago
rwa approved these changes 11 months ago
fnetX merged commit ed1ebc8b0d into master 11 months ago
Collaborator

Deployed and tested, I'm impressed by the redirect 😎

Deployed and tested, I'm impressed by the redirect 😎
rwa deleted branch issue-129 11 months ago
n commented 11 months ago
Poster
Collaborator

Credit goes to this comment :)

Credit goes to [this comment](https://github.com/11ty/eleventy/issues/510#issuecomment-824104799) :)

Reviewers

fnetX approved these changes 11 months ago
rwa approved these changes 11 months ago
The pull request has been merged as ed1ebc8b0d.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.