maybeInclude seems to not work (for me?)

[tunnckoCore]

I have maybeInclude('docs/API.md')

And when it not exists, it's okey.
But when exists it does not includes it.

Using ifExists helpers the things are working correctly.

ifExists('docs/API.md', include('docs/API.md'))

[jonschlinkert]

maybeInclude seems to not work (for me?)

That's why it's not called definitelyInclude lol. jk

@doowb or I can take a look and see what's happening.

[doowb]

It looks like 'docs/API.md' would need to be on the context for it to try to include it.

I'm not sure that's entirely correct logic. It might need to be something like:

if (opts[name] || utils.anyExists(name, this.app.cwd)) {
  const fn = this.app.getAsyncHelper(helperName);
  return fn.apply(this, arguments);
}

I think this will find includes that are already on the context and check the filesystem if they aren't.

[tunnckoCore]

It's okey to me to use ifExists anyway, i'm just reporting if it's not correct behavior.

Feel free to close if you both decide. ;]

(edit: side note... what you use for terminal recordings (and gifs)? any ideas? ;d)

[jonschlinkert]

what you use for terminal recordings (and gifs)? any ideas?

I use http://recordit.co/ sometimes. I'd like to find something that is better for terminal recordings though