Skip to content

feat(ci): add basic workflow and CMake entrypoint.#17

Closed
slfan1989 wants to merge 1 commit into
apache:mainfrom
slfan1989:paimon-cpp-16
Closed

feat(ci): add basic workflow and CMake entrypoint.#17
slfan1989 wants to merge 1 commit into
apache:mainfrom
slfan1989:paimon-cpp-16

Conversation

@slfan1989
Copy link
Copy Markdown
Contributor

Purpose

Linked issue: close #16

Add a basic GitHub Actions CI workflow for pull request checks.

This PR introduces:

  • A pre-commit job to run repository hygiene checks with pre-commit run --all-files.
  • A cmake job to verify that the project can be configured and built with CMake.
  • A minimal root CMakeLists.txt as the initial CMake entrypoint.

This is intended as the first CI step for the repository. Full unit test execution can be added incrementally after the CMake build/test targets and dependency setup become more complete.

Tests

Verified locally

API and Format

No.

Documentation

No user-facing documentation is added.

Generative AI tooling

@zjw1111
Copy link
Copy Markdown
Contributor

zjw1111 commented May 26, 2026

Thank you for your interest in and contribution to paimon-cpp.

We are currently in the process of migrating the codebase, and the CI pipeline will be improved step by step afterward. Merging new functionality at this stage may add extra complexity to the remaining migration work. You are very welcome to contribute to paimon-cpp again once the migration is completed.

Thanks again for your support and understanding!

@slfan1989
Copy link
Copy Markdown
Contributor Author

Thank you for your interest in and contribution to paimon-cpp.

We are currently in the process of migrating the codebase, and the CI pipeline will be improved step by step afterward. Merging new functionality at this stage may add extra complexity to the remaining migration work. You are very welcome to contribute to paimon-cpp again once the migration is completed.

Thanks again for your support and understanding!

@zjw1111 Thank you for the clarification and feedback! I understand that the project is currently undergoing codebase migration, and merging new functionality at this stage may add extra complexity to the migration work.

I will keep these changes for now and revisit them after the migration is completed and the CI pipeline becomes more stable, adapting them to the new code structure before contributing again.

Thanks again for your maintenance efforts, and I look forward to continuing to contribute to paimon-cpp in the future.

@slfan1989
Copy link
Copy Markdown
Contributor Author

@zjw1111 I will close the related PRs for now. Is there anything I can help with during the current migration stage?

@slfan1989 slfan1989 closed this May 26, 2026
@zjw1111
Copy link
Copy Markdown
Contributor

zjw1111 commented May 26, 2026

@zjw1111 I will close the related PRs for now. Is there anything I can help with during the current migration stage?

Thanks for asking! I’ll reach out if anything comes up where coordination or review would be helpful.

@lxy-9602
Copy link
Copy Markdown
Contributor

lxy-9602 commented May 26, 2026

BTW, our current main repository is still at https://github.com/alibaba/paimon-cpp. Since the migration to Apache may take some time, if you would like to try things out quickly or contribute new features, it might be easier to work with the original repository for now. We will continue to upstream all changes to Apache incrementally. Thanks again for your interest!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Add basic GitHub Actions workflow for pull request checks

3 participants