Skip to content

Updated calo analyzers to have additional printouts. Also, added IERC analyzer module#26

Merged
eflumerf merged 3 commits intodevelopfrom
pgirotti/IERC_updates
Feb 25, 2025
Merged

Updated calo analyzers to have additional printouts. Also, added IERC analyzer module#26
eflumerf merged 3 commits intodevelopfrom
pgirotti/IERC_updates

Conversation

@giro94
Copy link
Copy Markdown
Contributor

@giro94 giro94 commented Feb 25, 2025

I have disabled clang-format in this repo.

.clang-format Outdated
Cpp11BracedListStyle: true
DerivePointerAlignment: false
DisableFormat: false
DisableFormat: true
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do you want to disable clang formatting?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have to say that the clang reformatting Eric ran over this repo was quite shocking.

I don't like how it changed the appearance of the code.
I know that I can simply change the rules in this configuration file, but I will have to learn what to change to match my style. For now, I simply disabled it (I think)

rrivera747
rrivera747 previously approved these changes Feb 25, 2025
Copy link
Copy Markdown
Contributor

@rrivera747 rrivera747 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@giro94
Copy link
Copy Markdown
Contributor Author

giro94 commented Feb 25, 2025

PR can't be pulled because of failing checks?

@eflumerf eflumerf merged commit ba3803e into develop Feb 25, 2025
5 of 6 checks passed
@eflumerf eflumerf deleted the pgirotti/IERC_updates branch February 25, 2025 22:38
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.

3 participants