Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the pre-commit workflow #449

Merged
merged 7 commits into from
Jul 13, 2023
Merged

Conversation

tmadlener
Copy link
Collaborator

@tmadlener tmadlener commented Jul 13, 2023

BEGINRELEASENOTES

  • Fix the pre-commit workflow by making it run on top of the key4hep nightlies that come with a recent enough root version to be able to work with the RNTuple addition (Add support for the new RNTuple format #395)
  • Fix a few minor complaints from newer versions of clang-format, clang-tidy and pylint
  • Enable building the RNTuple backend for more workflows (anything that comes with a new enough ROOT essentially).

ENDRELEASENOTES

@hegner
Copy link
Collaborator

hegner commented Jul 13, 2023

Great. Waiting to see what the test will say :-)

@hegner
Copy link
Collaborator

hegner commented Jul 13, 2023

@tmadlener - I assume it only takes effect after merging :-(

@tmadlener
Copy link
Collaborator Author

No, it should take into account the state of this branch. I am not entirely sure why pylint starts complaining now, could be a minor version mismatch between before and now. I am having a look at this

@tmadlener
Copy link
Collaborator Author

OK. Now everything should work. Let's see if CI agrees.

@hegner hegner merged commit eb9276b into AIDASoft:master Jul 13, 2023
@hegner
Copy link
Collaborator

hegner commented Jul 13, 2023

thanks!

hegner pushed a commit to hegner/podio that referenced this pull request Jul 27, 2023
* Switch to a newer LCG stack with root 6.28/00

* Try with key4hep nightlies

* Allow git to run necessary commands for pre-commit

* Fix clang-format complaints

* Fix clang-tidy complaints

* Reorder some imports to make pylint happy again

* Enbale building RNTuple support in more workflows
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