nbdev_update_lib errors out when fname not supplied

[RenzoBenemerito]

nbdev/nbdev/sync.py

Line 126 in f8d6ed2

if fname.endswith('.ipynb'): raise ValueError("`nbdev_update_lib` operates on .py files. If you wish to convert notebooks instead, see `nbdev_build_lib`.")

When doing nbdev_update_lib without specifying the optional fname argument, it fails with an error as shown below instead of traversing the lib path directory.

[RenzoBenemerito]

I think its caused by this commit https://github.com/fastai/nbdev/commit/f2f4c32c385d3dc7920edd723d73f704bc00255d#diff-d8cd2fcd987f7884b494f7373cf50eb48cd2f925120114e153b08f779e326f95

[hamelsmu]

edit: found the link, but I cannot reproduce your error. I'm pretty sure you need to ugprade nbdev

pip install -U git+https://github.com/fastai/nbdev.git

[hamelsmu]

If upgrading doesn't fix it for you will be happy to reopen this issue please let me know.

[RenzoBenemerito]

If upgrading doesn't fix it for you will be happy to reopen this issue please let me know.

The problem still persists, I already have the latest version of nbdev

I think we need to add a None checker before the if statement in the code. If the default of fname is None and you execute the if statement fname.endswith, it will cause the error seen above.

[hamelsmu]

Thanks. I was testing the wrong command! I was testing nbdev_build_lib instead 🤦🏽 sorry. This is going to be fixed shortly via #416

[RenzoBenemerito]

Thanks. I was testing the wrong command! I was testing nbdev_build_lib instead 🤦🏽 sorry. This is going to be fixed shortly via #416

Thank you sir! 💯