Skip to content
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

[#5473] improvement(CI): Change CI build file to add bundles files into the build task. #5479

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

yuqi1129
Copy link
Contributor

@yuqi1129 yuqi1129 commented Nov 6, 2024

What changes were proposed in this pull request?

Add the bundles and their sub-modules to the changes listening list.

Why are the changes needed?

Bundles are newly added modules and should be added to the changes listening list.

Fix: #5473

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

CI.

@yuqi1129 yuqi1129 self-assigned this Nov 6, 2024
@yuqi1129 yuqi1129 requested a review from mchades November 6, 2024 06:07
Copy link
Contributor

@mchades mchades left a comment

Choose a reason for hiding this comment

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

LGTM.

@jerryshao jerryshao added the branch-0.7 Automatically cherry-pick commit to branch-0.7 label Nov 6, 2024
@jerryshao jerryshao merged commit 06aa939 into apache:main Nov 6, 2024
26 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 6, 2024
…to the build task. (#5479)

### What changes were proposed in this pull request?


Add the bundles and their sub-modules to the changes listening list. 

### Why are the changes needed?

Bundles are newly added modules and should be added to the changes
listening list.

Fix: #5473 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

CI.
mplmoknijb pushed a commit to mplmoknijb/gravitino that referenced this pull request Nov 6, 2024
…les into the build task. (apache#5479)

### What changes were proposed in this pull request?


Add the bundles and their sub-modules to the changes listening list. 

### Why are the changes needed?

Bundles are newly added modules and should be added to the changes
listening list.

Fix: apache#5473 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

CI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch-0.7 Automatically cherry-pick commit to branch-0.7
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Improvement] Build task will not compile bundles modules and it's sub-modules.
3 participants