This repository has been archived by the owner. It is now read-only.

Keep dependency options and convert :type to :extension #4

Merged
merged 1 commit into from Apr 4, 2012

Conversation

Projects
None yet
2 participants
@xeqi
Contributor

xeqi commented Apr 4, 2012

Fixes issues raised in technomancy/leiningen#457. This includes #3 and the loss of dep-spec options.

technomancy added a commit that referenced this pull request Apr 4, 2012

Merge pull request #4 from xeqi/dependency-options
Keep dependency options and convert :type to :extension

@technomancy technomancy merged commit 9b415b9 into technomancy:master Apr 4, 2012

@technomancy

This comment has been minimized.

Show comment Hide comment
@technomancy

technomancy Apr 4, 2012

Owner

Fantastic; thanks. Sorry, I somehow missed #3; I suppose this supersedes it?

Owner

technomancy commented Apr 4, 2012

Fantastic; thanks. Sorry, I somehow missed #3; I suppose this supersedes it?

@xeqi

This comment has been minimized.

Show comment Hide comment
@xeqi

xeqi Apr 4, 2012

Contributor

Yes. Is there a way to attach pull request to an existing issue?

Contributor

xeqi commented Apr 4, 2012

Yes. Is there a way to attach pull request to an existing issue?

@technomancy

This comment has been minimized.

Show comment Hide comment
@technomancy

technomancy Apr 4, 2012

Owner

You can put "Fixes #3" in the commit message; that's the only way I know.

Owner

technomancy commented Apr 4, 2012

You can put "Fixes #3" in the commit message; that's the only way I know.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.