FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Don't try to download windows exe/msi and mac universal pkg if it doesn't exists for python version requested by jmarrec · Pull Request #276 · actions/python-versions · GitHub

Don't try to download windows exe/msi and mac universal pkg if it doesn't exists for python version requested - #276

Open
jmarrec wants to merge 2 commits into
actions:mainfrom
jmarrec:windows_pkg_exists
Open

Don't try to download windows exe/msi and mac universal pkg if it doesn't exists for python version requested#276
jmarrec wants to merge 2 commits into
actions:mainfrom
jmarrec:windows_pkg_exists

Conversation

jmarrec commented May 29, 2024

Copy link
Copy Markdown

Only some minor versions of python have an actual windows .exe / .msi installer.

eg:

jmarrec requested a review from a team as a code owner May 29, 2024 05:30
jmarrec changed the title Don't try to continue for windows if download link for exe/msi doesn't exists for python version requested Don't try to download windows exe/msi and mac universal pkg if it doesn't exists for python version requested May 29, 2024

mayeut commented Jun 1, 2024
edited
Loading

Copy link
Copy Markdown
Contributor

Rather than silently ignore some network failures that would lead to the version being skipped, the json returned by https://www.python.org/api/v2/downloads/release/?is_published=true/https://www.python.org/api/v2/downloads/release_file/?release=... should be parsed to check if the package exists or not. This shall probably be done while generating the build matrix.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL