Skip to content
This repository was archived by the owner on Sep 11, 2020. It is now read-only.

new plumbing package #118

Merged
merged 5 commits into from
Nov 8, 2016
Merged

new plumbing package #118

merged 5 commits into from
Nov 8, 2016

Conversation

mcuadros
Copy link
Contributor

@mcuadros mcuadros commented Nov 7, 2016

This introduce a new plumbing package, this package contains all the core objects, and formats and clients packages. The goal is to introduce the logic division in git of plumbing and porcelain. All the package outside of the package plumbing are consider porcelain.

The main reason is because after think about where put the new IndexStorer, I figure out, that the Index, Objects, clients are consider plumbing, and instead of having a cross reference between two package of same level as core and formats/index

Copy link
Contributor

@alcortesm alcortesm left a comment

Choose a reason for hiding this comment

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

👍

@mcuadros
Copy link
Contributor Author

mcuadros commented Nov 8, 2016

PTAL @smola

"gopkg.in/src-d/go-git.v4/core"
"gopkg.in/src-d/go-git.v4/plumbing/clients"
"gopkg.in/src-d/go-git.v4/plumbing/clients/common"
"gopkg.in/src-d/go-git.v4/plumbing"
Copy link
Collaborator

Choose a reason for hiding this comment

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

gofmt missing?

@mcuadros mcuadros merged commit f49096e into src-d:master Nov 8, 2016
@mcuadros mcuadros deleted the plumbing branch December 13, 2016 10:13
mcuadros added a commit that referenced this pull request Jan 31, 2017
* plumbing: now core was renamed to core, and formats and clients moved inside
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants