Skip to content

Conversation

@navneetkumar231295
Copy link
Contributor

Dear Experts,

In this pull request I have fixed the O2-macos error.

@github-actions
Copy link

github-actions bot commented Jan 7, 2026

O2 linter results: ❌ 2 errors, ⚠️ 1 warnings, 🔕 0 disabled

@dmallick2 dmallick2 enabled auto-merge (squash) January 7, 2026 02:21
dmallick2
dmallick2 previously approved these changes Jan 7, 2026
auto-merge was automatically disabled January 7, 2026 02:56

Head branch was pushed to by a user without write access

@alibuild
Copy link
Collaborator

alibuild commented Jan 7, 2026

Error while checking build/O2Physics/o2 for 45af0b4 at 2026-01-07 04:01:

## sw/BUILD/O2Physics-latest/log
CMake Error at cmake/O2PhysicsAddExecutable.cmake:61 (message):

Full log here.

Fix O2 linter: align workflow file and struct naming

Fix O2 linter: align workflow file and struct naming

Fix O2 linter: align workflow file, struct, and CMake naming
@navneetkumar231295 navneetkumar231295 force-pushed the spin-alignment-measurements branch from 45af0b4 to fdc7123 Compare January 7, 2026 04:57
@navneetkumar231295
Copy link
Contributor Author

I attempted to fix the o2-linter error, which required renaming the file from chargedkstaranalysis.cxx to chargedKstarAnalysis.cxx to follow the lowerUpperCase naming convention. During this process, a few changes got mixed up. Hence, I am closing this pull request and will submit a fresh one.

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

Labels

Development

Successfully merging this pull request may close these issues.

4 participants