Skip to content

ci: update publish tarball logic#8142

Merged
mircea-c merged 2 commits intoanza-xyz:masterfrom
mircea-c:build/upload-ci-artifacts
Sep 23, 2025
Merged

ci: update publish tarball logic#8142
mircea-c merged 2 commits intoanza-xyz:masterfrom
mircea-c:build/upload-ci-artifacts

Conversation

@mircea-c
Copy link
Copy Markdown

@mircea-c mircea-c commented Sep 22, 2025

Problem

publish-tarball.sh and upload-ci-artifact.sh both contain logic to upload artefacts that is not being used anymore.

Summary of Changes

  • remove unused s3 and appveyor upload logic

Fixes #

closes https://github.com/anza-xyz/devops/issues/689

@mircea-c mircea-c requested a review from a team September 22, 2025 21:44
@mircea-c mircea-c self-assigned this Sep 22, 2025
@mircea-c mircea-c force-pushed the build/upload-ci-artifacts branch 4 times, most recently from 12042cd to 491e40e Compare September 22, 2025 21:51
Comment thread ci/publish-tarball.sh
Comment thread ci/publish-tarball.sh Outdated
Comment thread ci/upload-ci-artifact.sh
@mircea-c mircea-c force-pushed the build/upload-ci-artifacts branch from 491e40e to 1e964dc Compare September 23, 2025 15:10
@mircea-c mircea-c requested a review from yihau September 23, 2025 15:12
@mircea-c
Copy link
Copy Markdown
Author

@yihau I've removed the upload logic and will push it in another PR as a separate step.

@mircea-c mircea-c requested a review from levsha September 23, 2025 15:20
@mircea-c mircea-c merged commit 31f7f09 into anza-xyz:master Sep 23, 2025
33 checks passed
@mircea-c mircea-c deleted the build/upload-ci-artifacts branch September 23, 2025 16:14
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.

3 participants