Sober7135 opened a new issue, #894: URL: https://github.com/apache/incubator-graphar/issues/894
### Describe the bug, including details regarding any error messages, version, and platform. ## Repository policy [CONTRIBUTING.md](https://github.com/apache/incubator-graphar/blob/main/CONTRIBUTING.md) states that all pull requests are squash merged. ## Current behavior and Expected behavior ### Current behavior #876 validates a merge commit https://github.com/apache/incubator-graphar/blob/d535a437667c6e2bf28f7f376930b13ed5aed63d/.github/workflows/ci.yml#L105-L109 See: https://github.com/apache/incubator-graphar/actions/runs/22542566764/job/65299867120?pr=892 In this action, I added `cat .git/COMMIT_EDITMSG`, the output is ``` Merge 8255be0ba41d4142ecaae5296aaa79c25f3e639a into d535a437667c6e2bf28f7f376930b13ed5aed63d ``` And see also: - actions/checkout 's docs: https://github.com/actions/checkout?tab=readme-ov-file#checkout-pull-request-head-commit-instead-of-merge-commit - stackoverflow discussion: https://stackoverflow.com/questions/62334460/git-history-in-a-github-action ### Expected behavior Under squash merge, the meaningful metadata for what lands in main is the **PR title** / **final squash commit message**. And I think we should add this CI for all PRs not just C++ PR ### Component(s) Developer Tools -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
