Skip to content

Conversation

@Yicong-Huang
Copy link
Contributor

@Yicong-Huang Yicong-Huang commented Jul 2, 2025

The amber artifact has been named as "texera". This PR changes it to "amber".

@Yicong-Huang Yicong-Huang requested a review from bobbai00 July 2, 2025 07:01
@Yicong-Huang Yicong-Huang self-assigned this Jul 2, 2025
@Yicong-Huang Yicong-Huang added the config Pull requests that update configuration label Jul 2, 2025
@Yicong-Huang Yicong-Huang changed the title chore: Rename Amber artifact chore: rename Amber artifact Jul 2, 2025
@Yicong-Huang Yicong-Huang changed the title chore: rename Amber artifact chore(build): rename Amber artifact Jul 2, 2025
@bobbai00
Copy link
Contributor

bobbai00 commented Jul 3, 2025

I will wait for #3516 to be merged, update this PR a bit, and approve

@Yicong-Huang Yicong-Huang force-pushed the yicong-modify-artifact-version-1.0.0 branch from 5d5f1b6 to 75a5145 Compare July 4, 2025 14:06
Base automatically changed from yicong-modify-artifact-version-1.0.0 to master July 4, 2025 14:12
@Yicong-Huang
Copy link
Contributor Author

@bobbai00 please go ahead update it.

@Yicong-Huang Yicong-Huang changed the title chore(build): rename Amber artifact build: rename Amber artifact Jul 6, 2025
Signed-off-by: Yicong Huang <17627829+Yicong-Huang@users.noreply.github.com>
@Yicong-Huang Yicong-Huang changed the title build: rename Amber artifact not- a valid name: rename Amber artifact Jul 6, 2025
@Yicong-Huang Yicong-Huang changed the title not- a valid name: rename Amber artifact build(amber): rename Amber artifact Jul 6, 2025
Copilot AI review requested due to automatic review settings October 3, 2025 05:07
@Yicong-Huang
Copy link
Contributor Author

@bobbai00 can I merge this PR?

Copy link
Contributor

Copilot AI left a 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.

@Yicong-Huang
Copy link
Contributor Author

closed as we have done it in #3878.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config Pull requests that update configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants