Fork me on GitHub
#duct
<
2018-10-13
>
scottlowe21:10:30

@weavejester With reference to the latest version of the Duct UPGRADING.md file. I’ve tried the example for the {:duct.profile/migrations #duct/include "migrations"} on a new bare bones Duct project. I added the example derivation of {:duct.profile/migrations [:duct.profile/base]} to duct_hierarchy.edn.

scottlowe21:10:09

The result is the prep fails with: Ambiguous key: :duct.profile/base. Found multiple candidates: :duct.profile/base, :duct.profile/migrations.

weavejester21:10:32

Oh, that looks like a bug

scottlowe21:10:08

I’m so glad that you’ve said that - because I’ve been looking at this for some time and can’t get it to work! 🙂

weavejester21:10:52

Sorry you ran into that. I should be able to fix it tomorrow. Would you mind opening up an issue on it?

scottlowe21:10:25

Yes, of course, I will open an issue now. I just wanted to get a sanity check before filing an issue.

weavejester21:10:53

0.11.0 is still in alpha, so you hit something I clearly hadn't tested thoroughly enough.

scottlowe21:10:39

I totally appreciate that it’s still in alpha. I’m just making some little test apps ahead of time in preparation for upgrading our real codebase.

scottlowe21:10:51

No pressure intended!

weavejester21:10:02

Just to check: you said you included it on "a new bare bones Duct project"

weavejester21:10:16

Did you create that project with lein new duct-alpha...?

scottlowe21:10:21

I used the alpha template, yes.

weavejester21:10:29

Okay, good 🙂

scottlowe21:10:14

I’ll file the issue with these details. You’ll clearly find out quickly if I’ve done something daft (or not). Thanks again.

weavejester21:10:05

Thanks for taking the time to fill it out. Though just from the error message it looks like I've used the wrong function in prep somehow.