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

Mocketry Install #5

Closed
ClementMastin opened this issue May 17, 2017 · 6 comments
Closed

Mocketry Install #5

ClementMastin opened this issue May 17, 2017 · 6 comments

Comments

@ClementMastin
Copy link
Contributor

It could be good if when you load Clap, it also load Mocketry because of the tests

@cdlm
Copy link
Collaborator

cdlm commented May 17, 2017

Doesn't the baseline already do it ?

@ClementMastin
Copy link
Contributor Author

I tested it now and it doesn't install

@cdlm
Copy link
Collaborator

cdlm commented May 17, 2017

Ok… if you can make it load both packages by default, it would be good. However I think it should be possible to load Clap-Core alone (e.g. in a deployment scenario).

@cdlm
Copy link
Collaborator

cdlm commented May 22, 2017

Works for me…

@ClementMastin
Copy link
Contributor Author

I didn't load Clap with what the Readme indicates.I load it with Iceberg so it's maybe a problem coming from Iceberg.

@cdlm cdlm closed this as completed in 44b9286 May 22, 2017
@cdlm
Copy link
Collaborator

cdlm commented May 22, 2017

Ah then it's a case of PEBCAK :p

Iceberg has no notion of dependency so if you asked it to load Clap-Core it loaded only that. That's what's the baseline is for: it specifies what groups of packages are interesting and in which order to load them.

cdlm added a commit that referenced this issue Nov 16, 2017
Load everything by default (for development). To avoid loading tests and
dependencies, load the `core` group.

Closes #5
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

No branches or pull requests

2 participants