Right now,

from

here expects a directory, not a file

hence the

copy

wrapper, but I should probably fix that

Hence:

@Directive
    fun copy(from: From, to: To) {
        with(File(from.path)) {
            includedFiles.add(IncludedFile(from, to, listOf(IFileSpec.FileSpec(name))))
        }
    }

There's a nasty type signature conflict I need to resolve for your suggestion to work

because there's both

include(From, To, vararg String)

and

include(From, To, vararg IFileSpec)

. Obviously,

kotlinc

can't pick one if you only pass two parameters

The implementation that I posted earlier was just

include(to, vararg files)

, that would have worked, but I think what you're saying make more sense.

Although, I still think

collect()

should return

List<File>

and that would solve your problem.

Sigh, I need to go back to Gradle cause of that fatJar bug

I miss the auto complete already ;-;

@redrield It's on my list, should get to it soon

@ethauvin

collect

returns

List<File>

in

master

right now

Actually it's even in 1.0.51

@User uploaded a file:

Who needs release notes anyway?

It looks like 1.0.49, 1.0.50 and 1.0.51 all point to the same commit?

git tag wise?

I totally cheat on release notes. I usually just do something like

git log --oneline --no-merges --format=" * %s"

@cedric You did see the other issues I mentioned about

copy()

not actually copying, etc. Or do I need to open a new issue? Don't think I can re-open the old one where I commented on.

I saw the new one, it's on my list

You can't reopen issues?

Not if you closed it, I think I can if I closed it, not sure.

Don't see a re-open option.

I just made you a collaborator. Not quite sure what kind of power I just gave you but I trust you'll use it well 🙂

uh-oh, now I gotta be careful, not my reckless usual self. 😉

Performance review is every quarter.

Payday on the 4th, like LeEco? 😉

BURN!

Still don't see a way to re-open it, so I'm not that powerful, I guess.

That's very odd, link?