Skip to content
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.

Sign up for GitHub

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

Jump to bottom

csv-to-docs: warn if json_docs dir is not empty when running #26

Closed
alxndrsn opened this issue Oct 12, 2017 · 1 comment · Fixed by #603
Closed

csv-to-docs: warn if json_docs dir is not empty when running #26

alxndrsn opened this issue Oct 12, 2017 · 1 comment · Fixed by #603
Assignees
@sugat009
Labels
Help wanted Good for first time contributions released

Comments

@alxndrsn
Copy link
Contributor

alxndrsn commented Oct 12, 2017 •
edited 

There are already docs in ${jsonDocsDir}.
New json files will be created along side these existing docs.
Are you sure you wish to continue?  [y/N]
@alxndrsn alxndrsn self-assigned this Oct 23, 2017
@alxndrsn alxndrsn changed the title Warn if json_docs dir is not empty when running csv-to-docs csv-to-docs: warn if json_docs dir is not empty when running Oct 23, 2017
@garethbowen garethbowen added the Help wanted Good for first time contributions label Apr 17, 2019
@sugat009 sugat009 self-assigned this Jan 23, 2024
sugat009 added a commit that referenced this issue Jan 24, 2024
@sugat009
fix(#26): add warning if jsonDir contains file when running csv-to-do…
f7440c6 
…c command
sugat009 added a commit that referenced this issue Jan 24, 2024
@sugat009
fix(#26): add warning if jsonDir contains file when running csv-to-do…
9ac25a8 
…c command
@sugat009 sugat009 linked a pull request Jan 24, 2024 that will close this issue
fix(#26): add warning if jsonDir contains file when running csv-to-doc command #603
Merged
@garethbowen garethbowen mentioned this issue Jan 24, 2024
Merged
sugat009 added a commit that referenced this issue Jan 25, 2024
@sugat009
fix(#26): refactor the checking if directory exists and raising warning
75966a7 
into a separate function and write unit tests for it
sugat009 added a commit that referenced this issue Jan 28, 2024
@sugat009
fix(#26): remove unnecessary comments and change fs_ to fs.fs in
4a5dd9a 
sync-fs.spec.js
sugat009 added a commit that referenced this issue Feb 2, 2024
@sugat009
fix(#26): move warnIfDirectoryIsNotEmpty to sync-fs.js
3fa0392
sugat009 added a commit that referenced this issue Feb 5, 2024
@sugat009
fix(#26): add warning if jsonDir contains file when running csv-to-do…
c8d7134 
…c command
sugat009 added a commit that referenced this issue Feb 5, 2024
@sugat009
fix(#26): add warning if jsonDir contains file when running csv-to-do…
e445c9b 
…c command (#603)
@medic-ci
Copy link
Collaborator

medic-ci commented Feb 5, 2024

🎉 This issue has been resolved in version 3.21.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sugat009 sugat009

Labels
Help wanted Good for first time contributions released
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(#26): add warning if jsonDir contains file when running csv-to-doc command medic/cht-conf
4 participants
@alxndrsn @garethbowen @medic-ci @sugat009