Skip to content

[VL][INFRA] Improve build bundle package workflow#9404

Merged
zhztheplayer merged 1 commit intoapache:mainfrom
wForget:main
Apr 24, 2025
Merged

[VL][INFRA] Improve build bundle package workflow#9404
zhztheplayer merged 1 commit intoapache:mainfrom
wForget:main

Conversation

@wForget
Copy link
Copy Markdown
Member

@wForget wForget commented Apr 23, 2025

What changes were proposed in this pull request?

Improve build bundle package workflow:

  • Add repo ref parameter
  • Upgrade upload-artifact version to v4 (v3 has been deprecated: Error: Missing download info for actions/upload-artifact@v3)
  • Upgrade download-artifact version to v4 (v3 has been deprecated: Missing download info for actions/download-artifact@v3)

How was this patch tested?

https://github.com/wForget/gluten/actions/runs/14631773617

image

@github-actions github-actions bot added the INFRA label Apr 23, 2025
@github-actions
Copy link
Copy Markdown

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/apache/incubator-gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

@wForget wForget changed the title [VL] Add repo ref and upgrade upload-artifact version for build bundle package workflow [VL][INFRA] Add repo ref and upgrade upload-artifact version for build bundle package workflow Apr 23, 2025
@wForget wForget changed the title [VL][INFRA] Add repo ref and upgrade upload-artifact version for build bundle package workflow [VL][INFRA] Improve build bundle package workflow Apr 23, 2025
Comment on lines +37 to +40
ref:
description: 'Checkout repo ref to build'
required: true
default: 'main'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@zhztheplayer zhztheplayer merged commit ebf3ef8 into apache:main Apr 24, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants