Skip to content

Conversation

@thilinarmtb
Copy link
Collaborator

@thilinarmtb thilinarmtb commented Nov 7, 2025

And update GitHub CI workflows.

@thilinarmtb thilinarmtb force-pushed the cmake_format branch 2 times, most recently from 71587fa to 3b39e13 Compare November 7, 2025 20:02
@thilinarmtb thilinarmtb requested a review from Copilot November 7, 2025 20:22
Copy link

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 pull request applies cmake-format to standardize CMake file formatting across the entire OCCA project. The changes introduce a .cmake_format.json configuration file with an 80-character line width and 2-space indentation, add a new GitHub Actions workflow to enforce formatting on future changes, and reformat all existing CMake files according to these standards.

  • Adds .cmake_format.json configuration file with formatting rules
  • Adds .github/workflows/format.yml to check CMake formatting in CI
  • Reformats all CMakeLists.txt and .cmake files throughout the project

Reviewed Changes

Copilot reviewed 74 out of 77 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
.cmake_format.json New configuration file defining CMake formatting standards (80-char width, 2-space indent)
.github/workflows/format.yml New CI workflow to verify CMake formatting on push/PR
.github/workflows/cff-validator.yml Adds pull_request trigger for CITATION.cff validation
.github/workflows/build.yml Removes trailing blank line
CMakeLists.txt Reformatted according to cmake-format standards
bin/CMakeLists.txt Reformatted with consistent indentation and line wrapping
cmake/*.cmake All Find modules and helper scripts reformatted
src/**/CMakeLists.txt All source directory build files reformatted for consistency
tests/CMakeLists.txt Test configuration reformatted
examples/**/CMakeLists.txt All example build files reformatted

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@thilinarmtb thilinarmtb merged commit 11c7b22 into development Nov 7, 2025
4 checks passed
@thilinarmtb thilinarmtb deleted the cmake_format branch November 7, 2025 20:55
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.

2 participants