Skip to content

fix: do not include CPython source archive in unix artefacts - #287

Merged
v-HarithaVattikuti merged 1 commit into
actions:mainfrom
mayeut:unix-remove-sources
Mar 10, 2026
Merged

v-HarithaVattikuti merged 1 commit into
actions:mainfrom
mayeut:unix-remove-sources

Conversation

@mayeut

@mayeut mayeut commented Jun 11, 2024

Copy link
Copy Markdown
Contributor

CPython sources are currently included in release artefacts. This wastes storage & bandwidth.

@mayeut
mayeut requested a review from a team as a code owner June 11, 2024 18:57
@mayeut
mayeut force-pushed the unix-remove-sources branch from 6c1374f to 658cb37 Compare June 13, 2024 19:34
@mayeut
mayeut force-pushed the unix-remove-sources branch 3 times, most recently from 8298b0f to 88c81c0 Compare June 26, 2024 20:38
@mayeut
mayeut force-pushed the unix-remove-sources branch from 88c81c0 to 1bf4ef3 Compare September 8, 2024 12:48
@mayeut

mayeut commented Nov 10, 2024

Copy link
Copy Markdown
Contributor Author

@priyagupta108, @priya-kinthali, @HarithaVattikuti,

can you review this please ?

@mayeut
mayeut force-pushed the unix-remove-sources branch from 1bf4ef3 to 92aec99 Compare November 10, 2024 16:47
@mayeut
mayeut force-pushed the unix-remove-sources branch from 92aec99 to 2ff1187 Compare January 7, 2025 21:17
@mayeut
mayeut force-pushed the unix-remove-sources branch from 2ff1187 to 5029d47 Compare February 20, 2025 21:44
@mayeut
mayeut force-pushed the unix-remove-sources branch from 5029d47 to 59d7554 Compare April 24, 2025 06:13
@mayeut
mayeut force-pushed the unix-remove-sources branch from 59d7554 to dd2127d Compare June 4, 2025 05:26
Copilot AI review requested due to automatic review settings December 8, 2025 20:12
@mayeut
mayeut force-pushed the unix-remove-sources branch from dd2127d to a91749d Compare December 8, 2025 20:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a storage and bandwidth waste issue by preventing CPython source archives from being included in Unix release artifacts. The source archive is only needed during compilation and should not be part of the final artifact.

  • Changes the download location of CPython source archives from WorkFolderLocation (which is archived) to TempFolderLocation (which is excluded from artifacts)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mayeut
mayeut force-pushed the unix-remove-sources branch from a91749d to ee09270 Compare March 5, 2026 21:48
@v-HarithaVattikuti
v-HarithaVattikuti merged commit 2f93389 into actions:main Mar 10, 2026
23 of 24 checks passed
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.

5 participants