-
Notifications
You must be signed in to change notification settings - Fork 113
build(amber): rename Amber artifact #3518
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
I will wait for #3516 to be merged, update this PR a bit, and approve |
5d5f1b6 to
75a5145
Compare
|
@bobbai00 please go ahead update it. |
Signed-off-by: Yicong Huang <17627829+Yicong-Huang@users.noreply.github.com>
|
@bobbai00 can I merge this PR? |
There was a problem hiding this comment.
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 renames the Amber artifact from "texera" to "amber" to ensure consistent naming across the codebase.
- Updated the artifact name in the SBT build configuration from "texera" to "amber"
- Modified shell scripts to reference the new "amber-" artifact paths instead of "texera-"
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| core/amber/build.sbt | Changed the SBT project name from "texera" to "amber" |
| core/scripts/build-services.sh | Updated unzip and rm commands to use amber-* artifact paths |
| core/scripts/workflow-computing-unit.sh | Updated computing unit master binary paths to use amber-* directory |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
closed as we have done it in #3878. |
The amber artifact has been named as "texera". This PR changes it to "amber".