Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions packages.ini
Original file line number Diff line number Diff line change
Expand Up @@ -2010,6 +2010,7 @@ python_versions = <3.13
[sentry-forked-djangorestframework-stubs==3.15.3.post1]
[sentry-forked-djangorestframework-stubs==3.16.0.post1]
[sentry-forked-djangorestframework-stubs==3.16.1.post1]
[sentry-forked-djangorestframework-stubs==3.16.2.post1]

Comment on lines +2013 to 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Potential bug: Adding a package to packages.ini won't make it installable. The build process always runs incrementally and won't regenerate the index for this change, causing installation failures.
  • Description: The build process, specifically the make_index.py script, always runs in an incremental mode using the --previous-package-list-json flag. This incremental mode does not add new package entries from packages.ini to the index unless a new wheel is built. However, the build.py script will not build a new wheel for the added package sentry-forked-djangorestframework-stubs==3.16.2.post1 because it determines the package is already satisfied by existing wheels. Consequently, the new package entry will not be indexed, causing any downstream installation attempts for this specific version to fail with a package not found error.

  • Suggested fix: The build workflow needs a mechanism to trigger a full regeneration of the PyPI index when packages.ini is modified. This could involve conditionally removing the --previous-package-list-json flag from the make_index.py call to force a full rebuild when changes to packages.ini are detected.
    severity: 0.85, confidence: 0.95

Did we get this right? 👍 / 👎 to inform future reviews.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lmao. absolute garbage

[sentry-forked-email-reply-parser==0.5.12.post1]

Expand Down
Loading