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

ensuring that dependent dll's are not included in the package when us… #1543

Merged
merged 1 commit into from Mar 24, 2016

Conversation

Projects
None yet
3 participants
@pms1969
Contributor

pms1969 commented Mar 23, 2016

…ing include-referenced-projects

as per requested in (#1540). Separate PR in order to keep the history clean.

Paul Saunders
@pms1969

This comment has been minimized.

Show comment
Hide comment
@pms1969

pms1969 Mar 23, 2016

Contributor

I'm having a great deal of trouble running any tests locally at present. Probably just a work thing. I'll keep an eye on the build progress.

Contributor

pms1969 commented Mar 23, 2016

I'm having a great deal of trouble running any tests locally at present. Probably just a work thing. I'll keep an eye on the build progress.

@pms1969

This comment has been minimized.

Show comment
Hide comment
@pms1969

pms1969 Mar 23, 2016

Contributor

Is that a relevant failure on Travis?

Contributor

pms1969 commented Mar 23, 2016

Is that a relevant failure on Travis?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Mar 23, 2016

Member

nope. that's unrelated

Member

forki commented Mar 23, 2016

nope. that's unrelated

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Mar 23, 2016

Member

@kolektiv @inosik would this still work for you?

Member

forki commented Mar 23, 2016

@kolektiv @inosik would this still work for you?

@inosik

This comment has been minimized.

Show comment
Hide comment
@inosik

inosik Mar 23, 2016

Contributor

I haven't been using this parameter anyways, so I think it should be fine.

Contributor

inosik commented Mar 23, 2016

I haven't been using this parameter anyways, so I think it should be fine.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Mar 23, 2016

Member

ok let's try it and see what happens.

Member

forki commented Mar 23, 2016

ok let's try it and see what happens.

@forki forki merged commit cfd54d0 into fsprojects:master Mar 24, 2016

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment