Skip to content

Put wheel files first when uploading.#107

Merged
sigmavirus24 merged 7 commits intomasterfrom
unknown repository
Apr 29, 2015
Merged

Put wheel files first when uploading.#107
sigmavirus24 merged 7 commits intomasterfrom
unknown repository

Conversation

@tyrelsouza
Copy link
Copy Markdown
Contributor

@tyrelsouza tyrelsouza commented Apr 29, 2015

References #106

This groups wheel ".whl" files first for upload order so that the metadata can be displayed immediately while other files are still uploading.

  • Added a function upload.group_wheel_files_first
  • Added a two tests.
  • Ensured coverage for function

@tyrelsouza
Function to put wheel files first for upload.
88421dc 
Referencing #106

Put wheel files first for upload so that the meta data can be displayed
immediately.
@tyrelsouza
Don't bother grouping if it doesn't need to.
b75232a 
#106

Add a conditional to check if there's any need to group by wheel files, or if it
can just return the unprocessed list of dist files.
@tyrelsouza
Ensure coverage for my function
e452e5d
@tyrelsouza
Use groupby rather than a forloop.
dac5512
@tyrelsouza
Put itertools import alphabetically in the correct place
6bc8791
@tyrelsouza
Put back indentation removed by pycharm
80094bf
@tyrelsouza
Add self to authors
772dad5
@tyrelsouza
Copy link
Copy Markdown
Contributor Author

tyrelsouza commented Apr 29, 2015

I see what you did there 😎.

sigmavirus24 added a commit that referenced this pull request Apr 29, 2015
@sigmavirus24
Merge pull request #107 from tyrelsouza/wheel_first
66e967a 
Put wheel files first when uploading.
@sigmavirus24 sigmavirus24 merged commit 66e967a into pypa:master Apr 29, 2015
@sigmavirus24
Copy link
Copy Markdown
Member

sigmavirus24 commented Apr 29, 2015

✨ 🍰 ✨

@sigmavirus24 sigmavirus24 modified the milestone: next Sep 14, 2015
@jamadden jamadden mentioned this pull request Sep 22, 2017
Closed
@jsirois jsirois mentioned this pull request Oct 23, 2022
Open
@rgommers rgommers mentioned this pull request Apr 25, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

1.6.0

Development

Successfully merging this pull request may close these issues.

2 participants

@tyrelsouza @sigmavirus24
⚡