PASTAplus / web-x

The EDI Website Project
Apache License 2.0
1 stars 1 forks source link

Update environment.yml files #7

Closed clnsmth closed 2 years ago

clnsmth commented 2 years ago

Fixes an parsing issue in environment-win.yml, adds the undeclared 'config' dependency, and installs 'bugle' from GitHub

servilla commented 2 years ago

Hi @clnsmth,

Thanks for adding the bugle entry for GitHub in the conda env files. Odd, but I never add the git repos into the conda environment files; I add them manually to the working project. I like having them in there, so I will continue this practice. :-)

However, the config package added to the environment files is not correct. Instead, there is a config.py.template file in the web-x source that you need to copy over to config.py - it's a local module used for local configuration. The *.template approach keeps private stuff out of GitHub.

Lastly, was there a specific parsing error with the environment-win.yml version of the file?

clnsmth commented 2 years ago

Thanks @servilla. The specific parsing error for environment-win.yml:

ERROR:

while parsing a block mapping
  in "<unicode string>", line 1, column 1:
    name: web-x
    ^
expected <block end>, but found '<block mapping start>'
  in "<unicode string>", line 25, column 2:
     prefix: /home/servilla/anaconda3 ... 

I'll update the PR.

clnsmth commented 2 years ago

Hi @servilla. I've removed config from the environment.yml files. The updated PR is available for your review.