Skip to content

Set DUB config type to autodetect#281

Merged
PetarKirov merged 1 commit intodlang-community:masterfrom
wilzbach:set-config-type-to-auto
May 22, 2017
Merged

Set DUB config type to autodetect#281
PetarKirov merged 1 commit intodlang-community:masterfrom
wilzbach:set-config-type-to-auto

Conversation

@wilzbach
Copy link
Member

This simple change and a new preview release tag afterwards is needed to integrate dfmt into the DLang Tour (dlang-tour/core#516) (as a "Format" button).

Copy link
Member

@PetarKirov PetarKirov left a comment

Choose a reason for hiding this comment

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

LGTM, I just verified that dlang-tour + pr 516 builds with this fix ;)

@PetarKirov PetarKirov merged commit fef85e3 into dlang-community:master May 22, 2017
@wilzbach wilzbach deleted the set-config-type-to-auto branch May 22, 2017 17:28
@PetarKirov
Copy link
Member

PetarKirov commented May 22, 2017

@wilzbach I've just pushed the 0.5.0 tag.

@wilzbach
Copy link
Member Author

Thanks a lot for the quick review & merge, guys. Very appreciated!

@ghost
Copy link

ghost commented May 23, 2017

why ? dfmt is well an executable after all.

@wilzbach
Copy link
Member Author

why ? dfmt is well an executable after all.

And it stays to be, but "autodetect" is a DUB feature which will change the build type to "library" if it's included as dependency to another project as the DLang Tour where import dfmt.formatter is called.

@ghost
Copy link

ghost commented May 23, 2017

Thanks for the clarification.

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