caffeine-addictt / video-manager

CLI tool for downloading and managing videos ⭐️ Star to support our work!
MIT License
7 stars 1 forks source link

[Bug] Configuration file in PWD not loaded #21

Closed caffeine-addictt closed 5 months ago

caffeine-addictt commented 5 months ago

Bug report

Your issue may already be reported! Please check out our active issues before creating one.

Current Behavior

Consider this typology

home/
| |_ .video-manager (dir: ~/Videos)
[...]/my-project/
  |_ .video-manager (dir: .)

When a command is executed in the my-project/ directory, the working directory resolved is ~/Videos.

Expected Behavior

The resolved directory should use [...]/my-project/.video-manager's dir: . and resolve to the current working directory.

Steps to Reproduce (for bugs)

  1. Create a .video-manager file in $HOME/
  2. Configure dir to a directory
  3. Switch to any other directory and redo step 1 in that new directory
  4. This time configure dir to .
  5. Run any command that interacts with the directory like list or get
  6. Notice that it will use the default ~/Videos instead of the configured dir

Context

With debug on, I can confirm that the configuration file is resolving correctly at the PWD. This looks to be an issue with using the values from the loaded configuration file

Your Environment