Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed logging check that was causing dependency fetching to fail #3009

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

DanBeard
Copy link

@DanBeard DanBeard commented Oct 3, 2022

Thanks taking the time to contribute. This template should help guide you through the process of creating a pull request for review. Please erase any part of this template that is not relevant to your pull request:

  • Have you followed the guidelines for contributors?
  • Have you checked to ensure there aren't other open pull requests on the repository for a similar change?
  • Is there a corresponding ticket for your pull request?
  • Have you written new tests for your changes?
  • Have you successfully run the project with your changes locally?

Quick fix for #3008 . source_info is nil since Pypi removed $["releases"] from the JSON API response. Commenting out this line stops the error from being thrown and allows dependencies to be grabbed for projects in my local env.

Unsure when the Pypi API was actually changed. Recent Pypi packages may need to be re-evaluated to ensure dependency information in libraries,io is correct.

Co-authored-by: Tieg Zaharia <tieg.zaharia@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants