Add warning at delete contents for Linux users on @fedify/init - #997
Add warning at delete contents for Linux users on @fedify/init#997lego37yoon wants to merge 5 commits into
@fedify/init#997Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
As @dahlia fixed formatting issues on writing changes, I did following actions:
Pre-commit hook also passed after this rebase. |
Summary
@fedify/initproject initialization asks users to use non-empty directory, and confirm to move existing contents to Trash. However, its behavior actually permanently delete all of existing contents without back up for Linux or UNIX-like system users.This pull request notices users who are using Linux or UNIX-like systems that this action deletes existing data permanently. Furthermore, refactored messages and action structure in
dir.tsfile to easily add conditions or supported systems.fedify init'move the contents to the trash' actually remove contents permanently #989@fedify/init#992Test Plan
mise run check-each initmise run test-each init