amplify-cli icon indicating copy to clipboard operation
amplify-cli copied to clipboard

fix(cli): fixed addPluginDirectory

Open NakZMichael opened this issue 4 years ago • 0 comments

Fix to prevent adding a directory that is already included in pluginDirectories.

Description of changes

When I ran amplify plugin configure -> select scannable plugin directories -> select add , if I specified a directory that was already included, it was registered as a duplicate.

I thought this behavior is maybe undesirable.

Issue #, if available

Description of how you validated changes

I don't know how to test automatically a validator function in inquirer.prompt 😢

  • manual test

Checklist

  • [x] PR description included
  • [x] yarn test passes
  • [ ] Tests are changed or added
  • [ ] Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

NakZMichael avatar Sep 13 '21 19:09 NakZMichael