Skip to content

Conversation

@Yicong-Huang
Copy link
Contributor

This folder is no longer needed.

resolves #3879

@Copilot Copilot AI review requested due to automatic review settings October 12, 2025 18:27
@Yicong-Huang Yicong-Huang changed the base branch from main to chore/relocate-amber October 12, 2025 18:28
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 removes the core/log folder and updates all references to reflect changes in the project structure. The changes involve removing the core directory prefix from paths and updating build scripts, documentation, and CI configuration to align with the new structure.

  • Removed core directory references from file paths throughout the codebase
  • Updated build and deployment script paths in Docker configuration
  • Modified CI workflows to use new directory structure

Reviewed Changes

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Show a summary per file
File Description
amber/src/main/python/core/storage/iceberg/test_iceberg_document.py Updated directory path to remove core/ prefix
access-control-service/src/main/scala/org/apache/texera/service/AccessControlService.scala Replaced utility import with inline path construction
Dockerfile Updated script paths to use bin/ directory
CONTRIBUTING.md Updated documentation paths to reflect new structure
.github/workflows/github-action-build.yml Updated CI script and directory paths
.github/labeler.yml Updated labeling configuration paths

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot added ci changes related to CI docs Changes related to documentations service labels Oct 12, 2025
@Yicong-Huang Yicong-Huang self-assigned this Oct 12, 2025
@Yicong-Huang Yicong-Huang removed dependencies Pull requests that update a dependency file python ci changes related to CI docs Changes related to documentations service labels Oct 12, 2025
@Yicong-Huang Yicong-Huang requested a review from chenlica October 12, 2025 18:28
Base automatically changed from chore/relocate-amber to main October 12, 2025 23:14
Signed-off-by: Yicong Huang <17627829+Yicong-Huang@users.noreply.github.com>
@github-actions github-actions bot added dependencies Pull requests that update a dependency file service labels Oct 12, 2025
@github-actions github-actions bot removed dependencies Pull requests that update a dependency file service labels Oct 12, 2025
@Yicong-Huang Yicong-Huang enabled auto-merge (squash) October 12, 2025 23:28
@aglinxinyuan aglinxinyuan disabled auto-merge October 12, 2025 23:30
@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Oct 12, 2025
@github-actions github-actions bot removed the dependencies Pull requests that update a dependency file label Oct 12, 2025
@Yicong-Huang Yicong-Huang enabled auto-merge (squash) October 12, 2025 23:36
@Yicong-Huang Yicong-Huang added the ci changes related to CI label Oct 12, 2025
@Yicong-Huang Yicong-Huang merged commit e1aa6be into main Oct 12, 2025
11 checks passed
@Yicong-Huang Yicong-Huang deleted the chore/remove-core-log branch October 12, 2025 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci changes related to CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

remove core/log

3 participants