PaulMcInnis / JobFunnel

Scrape job websites into a single spreadsheet with no duplicates.
MIT License
1.81k stars 212 forks source link

log_path has an invalid value #58

Closed ikeos2 closed 4 years ago

ikeos2 commented 4 years ago

Issue Template

Description

the log path variable isn't being set. I tried setting the log_path in settings too, but not sure what the issue is.

Steps to Reproduce

  1. Install via pip
  2. Copy settings.yaml into ~/Virtualenv/Lib/site-packages/jobfunnel/config
  3. Run 'funnel'

Expected behavior

Funnel runs

Actual behavior

$ funnel
ConfigError: 'log_path' has an invalid value

Environment

ikeos2 commented 4 years ago

If I comment out lines 54 and 55 in config/validate.py, the script will run.

studentbrad commented 4 years ago

We will have to try reproducing this issue on Windows 10 using Python 3.7.3. You'll have to bear with us. We only have so much time. Thank you for bringing this to our attention. In the meantime, if you have any more information let us know here 😄

markkvdb commented 4 years ago

Thanks for pointing out this bug! The reason for the ConfigError is that I forgot about Windows users in the validation process 👎 . The regex to check for valid paths did not consider backward slashes (). This should be fixed in #62 but since I do not have a Windows system could you possibly check whether the changes fixes your problem?

Assuming you have downloaded the repository to your system this can be done by opening the relevant branch and install JobFunnel as

git checkout origin/logpath-fix
pip install -e .

Assuming you are in the JobFunnel directory. Let me know whether this works so that we can merge the pull request into the master.

markkvdb commented 4 years ago

Solved in #62