Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split functionality into five crates from one #301

Merged
merged 1 commit into from
Mar 17, 2016
Merged

Conversation

reset
Copy link
Collaborator

@reset reset commented Mar 17, 2016

  • bldr - the initial crate mostly unchanged (for now)
  • core - shared functionality between any bldr crate
  • depot - contains the depot server
  • depot-core - shared functionality between any depot components
  • depot-client - an http client to the depot server

This is a large code upheaval so I have left quite a bit of duplicate code in places since we have code inflight. There are largely no actual changes here aside from making the compiler happy with the new location of code paths when necessary. The depot crate contains the largest amount of actual changes.

The Depot crate now generates an executable artifact of it's own which contains a clap CLI. This is the majority of the new bits.

gif-keyboard-13225070687577517519

* bldr - the initial crate mostly unchanged (for now)
* core - shared functionality between any bldr crate
* depot - contains the depot server
* depot-core - shared functionality between any depot components
* depot-client - an http client to the depot server
@chef-delivery
Copy link
Contributor

This PR has passed 'Verify' and is ready for review and approval!
Use: '@delivery approve' when code review is complete.

@adamhjk
Copy link
Contributor

adamhjk commented Mar 17, 2016

Two questions:

  1. Are we ever going to publish these crates? If so, do we want their names to include 'habitat', or some variation of that?
  2. Do we want to have the binaries include some form of 'hab-'?

@reset
Copy link
Collaborator Author

reset commented Mar 17, 2016

  1. We'll for sure be publishing all of these crates
  2. Yes, but we should do that in a separate PR :)

@adamhjk
Copy link
Contributor

adamhjk commented Mar 17, 2016

gif-keyboard-17307583297617377202

@adamhjk
Copy link
Contributor

adamhjk commented Mar 17, 2016

@delivery approve

chef-delivery added a commit that referenced this pull request Mar 17, 2016
Merged change 378dd65a-b60a-4fec-a18c-6a960cb978a1

From review branch componentize into master

Signed-off-by: adam <[email protected]>
@chef-delivery chef-delivery merged commit bf61843 into master Mar 17, 2016
@chef-delivery
Copy link
Contributor

Change: 378dd65a-b60a-4fec-a18c-6a960cb978a1 approved by: @adamhjk

@chef-delivery chef-delivery deleted the componentize branch March 17, 2016 18:06
@chef-delivery
Copy link
Contributor

This PR has passed 'Acceptance' and is ready to be delivered!
Use: '@delivery deliver' when validated in acceptance.

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.

3 participants