Skip to content

Conversation

@jontsai
Copy link
Contributor

@jontsai jontsai commented Oct 30, 2025

Summary

  • Fixed package name in setup.py from altimate-datapilot-cli to altimate_datapilot_cli to comply with PyPI filename requirements

Problem

The tag-release workflow was failing with a 400 Bad Request error from PyPI:

ERROR    HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/        
         Filename 'altimate-datapilot-cli-0.1.1.tar.gz' is invalid, should be   
         'altimate_datapilot_cli-0.1.1.tar.gz'.

Solution

PyPI requires distribution filenames to use underscores (_) instead of hyphens (-). This change updates the package name in setup.py to meet this requirement.

Impact

  • Users can still install using either pip install altimate-datapilot-cli or pip install altimate_datapilot_cli because PyPI normalizes package names
  • This change only affects the distribution filename, not the user-facing install command
  • No breaking changes for existing users

Test Plan

  • Merge this PR
  • Create a new version bump PR with the version-bump label
  • Verify the tag-release workflow succeeds and uploads to PyPI

🤖 Generated with Claude Code


Important

Change package name in setup.py to use underscores, resolving PyPI filename error.

  • Behavior:
    • Change package name in setup.py from altimate-datapilot-cli to altimate_datapilot_cli to comply with PyPI requirements.
    • Resolves 400 Bad Request error during tag-release workflow due to invalid filename.
  • Impact:
    • Distribution filename now uses underscores, aligning with PyPI standards.
    • No impact on user-facing install command; users can still use hyphens or underscores.
    • No breaking changes for existing users.

This description was created by Ellipsis for 0985ae6. You can customize this summary. It will automatically update as commits are pushed.

Changed package name from 'altimate-datapilot-cli' to 'altimate_datapilot_cli' in setup.py to comply with PyPI's filename requirements. PyPI requires distribution filenames to use underscores, not hyphens.

This fixes the 400 Bad Request error during package upload: 'Filename altimate-datapilot-cli-0.1.1.tar.gz is invalid, should be altimate_datapilot_cli-0.1.1.tar.gz.'

Note: Users can still install using either 'pip install altimate-datapilot-cli' or 'pip install altimate_datapilot_cli' as PyPI normalizes package names.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 0985ae6 in 23 seconds. Click for details.
  • Reviewed 13 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. setup.py:15
  • Draft comment:
    Correct package name update to use underscores for PyPI compliance. Ensure any related docs or metadata references are updated as needed.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_JhniN0DhFozDWsDR

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@jontsai
Copy link
Contributor Author

jontsai commented Oct 30, 2025

Closing this PR to investigate further. Version 0.1.1 appears to already be on PyPI despite the workflow error.

@jontsai jontsai closed this Oct 30, 2025
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