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

Moving to setup.cfg + adding a .pyup.yml breaks WebUI #350

Closed cooperlees closed 5 years ago

cooperlees commented 5 years ago

I am unsure what my error is to cause the PyUP Web UI to render unusable (screenshot attached). I might try reversing the logic to only update: False on setup.cfg

I can't remove the .pyup.yml as I don't want setup.cfg to have it's dependencies updated / pinned.

What's my best way forward? This has also stopped any PRs being made. So I am sure my .yml is at fault here.

requirements:

assignees:

Screen Shot 2019-06-18 at 6 11 50 AM

cooperlees commented 5 years ago

I've tried adding and removing the pypa/bandersnatch repo via the WebUI but I always end up in the same state.

cooperlees commented 5 years ago

I give up. Going to try dependbot. If you ever care to fix this, that would be nice.

Jwomers commented 5 years ago

Hi Cooper, we're looking into this right now to see why you're getting that error.

Thanks for your patience! Justin

On Mon, 24 Jun 2019 at 14:24, Cooper Lees notifications@github.com wrote:

I give up. Going to try dependbot. If you ever care to fix this, that would be nice.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pyupio/pyup/issues/350?email_source=notifications&email_token=AAAV6QVRR32UMDZA3QFQKZDP4E3PXA5CNFSM4HY76V32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYOIX4Q#issuecomment-505187314, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAV6QSCGMJBUOYQAYKJHALP4E3PXANCNFSM4HY76V3Q .

Jwomers commented 5 years ago

Hi Cooper!

It looks like this might be your current master .yml file indentation. In the requirements section, there should be two spaces before each list item, and then within those each property is intended by 4 spaces. Assignees also need to be indented by a space. See our example in our docs: https://pyup.io/docs/bot/config/

Does this help?

On Mon, 24 Jun 2019 at 15:09, Justin Womersley justinwomersley@gmail.com wrote:

Hi Cooper, we're looking into this right now to see why you're getting that error.

Thanks for your patience! Justin

On Mon, 24 Jun 2019 at 14:24, Cooper Lees notifications@github.com wrote:

I give up. Going to try dependbot. If you ever care to fix this, that would be nice.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pyupio/pyup/issues/350?email_source=notifications&email_token=AAAV6QVRR32UMDZA3QFQKZDP4E3PXA5CNFSM4HY76V32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYOIX4Q#issuecomment-505187314, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAV6QSCGMJBUOYQAYKJHALP4E3PXANCNFSM4HY76V3Q .

cooperlees commented 5 years ago

Will try - Thought I'd already had that format (with https://github.com/pypa/bandersnatch/pull/269/files). Can we get it to work with yamllint format? That would take all the guessing out.

Lets see how just adding the spaces your request goes.

cooperlees commented 5 years ago

Landed that. Still does not work. I've tried reloading multiple times. I just get the same very informative (😢)

Error
Unknown Error

error.

Jwomers commented 5 years ago

Damn ok I will debug tomorrow. Thanks for trying that Cooper

On Mon, Jun 24, 2019 at 7:46 PM Cooper Lees notifications@github.com wrote:

Landed that. Still does not work. I've tried reloading multiple times. I just get the same very informative (😢)

Error

Unknown Error

error.

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/pyupio/pyup/issues/350?email_source=notifications&email_token=AAAV6QTXAZ524YCYY63K6JTP4GBJXA5CNFSM4HY76V32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYOZ5DA#issuecomment-505257612, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAV6QT2E6KZ6CZNGTKGTGTP4GBJXANCNFSM4HY76V3Q .

cooperlees commented 5 years ago

Ok, I removed everything I can live without and made sure my spaces matched the example exactly.

I now have 8 PRs to work my way through :)

I really do feel any valid .yaml / .yml should be accepted. Are there plans for that?

cooperlees commented 5 years ago

Well, https://github.com/pypa/bandersnatch/blob/master/.pyup.yml works, so going to close this. There are many other Issues open for standard YAML and / or more useful errors on the web interface. No need for this one.