Skip to content

Conversation

@Secrus
Copy link
Member

@Secrus Secrus commented Nov 28, 2022

Pull Request Check List

Resolves: long-standing TODO comment

  • Added tests for changed code. We don't test the metadata
    - [ ] Updated documentation for changed code. No changes needed

@neersighted neersighted force-pushed the uploader-provides-extra branch from 00df5e4 to 5cc6d23 Compare December 29, 2022 19:50
Copy link
Member

@radoering radoering left a comment

Choose a reason for hiding this comment

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

Can we add a test in test_uploader? Something like

def test_uploader(uploader: Uploader, fixture_dir: FixtureDirGetter) -> None:
    post_data = uploader.post_data(fixture_dir("distributions") / "demo-0.1.0-py2.py3-none-any.whl")
    assert post_data == ...

for instance?

@Secrus Secrus marked this pull request as draft March 11, 2024 08:42
@Secrus Secrus closed this Feb 9, 2026
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