Skip to content

Conversation

@singiamtel
Copy link
Contributor

I did this with some bash helper script, TBD if it's worth it to add to the CI

https://gist.github.com/singiamtel/9c728b4adeb4bee0a2c7d34d56d4a507

CC @vkucera @ktf

@github-actions
Copy link

github-actions bot commented Jun 3, 2025

O2 linter results: ❌ 2363 errors, ⚠️ 0 warnings, 🔕 44 disabled

@github-actions github-actions bot changed the title Import system libraries using angle brackets [Common,ALICE3,PWGLF,PWGUD,DPG,Tutorial] Import system libraries using angle brackets Jun 3, 2025
@njacazio njacazio enabled auto-merge (squash) June 3, 2025 11:39
Copy link
Collaborator

@vkucera vkucera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @singiamtel !
Approving just formally since PWGHF is clean. ;-)
I would be in favour of adding it in the CI.
@ddobrigk What do you think?

@njacazio njacazio merged commit fe97d6b into master Jun 6, 2025
19 of 22 checks passed
@njacazio njacazio deleted the fix_imports branch June 6, 2025 15:18
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Jun 11, 2025
hernasab pushed a commit to hernasab/O2Physics that referenced this pull request Jun 11, 2025
EmilGorm pushed a commit to EmilGorm/O2Physics that referenced this pull request Jun 12, 2025
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jun 12, 2025
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jun 14, 2025
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jul 16, 2025
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

9 participants