-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: check if folder is empty #22
Conversation
Nice. We'll work on getting this in once the lang branch is merged. |
@roblarsen @coliff Should |
This pull request introduces 1 alert when merging 63b404c into e3dfaba - view on LGTM.com new alerts:
|
This one is ready too. Although I still have a problem with testing.. I'd be happy for a help. Need to mock prompt node module somehow.. |
@vltansky do you want to open an issue that describes the question you have about testing? We can focus on fixing that and do "help wanted" if we need to. I'll work on merging this in. |
@roblarsen Do you want to merge this? And leave tests subject for separate future PR (as it not directly blocking this PR, but overall issue)? |
👍 |
Types of changes
Checklist:
Check if folder is empty
This branch is based on
lang
branch as it has a lot of changes includingprompts
module.