search

pyupio / pyup

A tool to update your project's dependencies on GitHub. Runs on pyup.io, comes with a command line interface.
https://pyup.io
MIT License
454 stars 67 forks source link

pyup chokes with 500 when facing .pyup.yml config file using strict syntax #346

Open ssbarnea opened 5 years ago

ssbarnea commented 5 years ago

I am not sure what YAML loader is used but pyup, but when I sorted all YAML warnings identified by yamllint inside the .pyup.yml I realised that https://pyup.io started to give 500 Internal Server Error instead of loading them.

Example: https://pyup.io/account/repos/github/ssbarnea/molecule/settings/

The config file can be seen at https://github.com/ssbarnea/molecule/commit/575c588328fc11c4e1e9eac42151605246f7ec70 and mainly is sorting two type of issues.

a) Use of standard boolean values true and false (lowercase) instead of the older syntax which was giving too much freedom (reported as truthy warnings by yamllint)

b) normalized identation.

pyup should not only accept these standard yaml files but should also document use of correct boolean addresses fro YAML files.

rafaelpivato commented 4 years ago

We certainly need to enhance our YAML handling. Thanks for reporting this.