Skip to content

Bump clang-format version to 10-12 #1

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

pgrete
Copy link

@pgrete pgrete commented Apr 28, 2022

Hi @JoshuaSBrown
I'm trying to bump the clang-format versions for parthenon.
If you're happy that this still lives in "your" repo, then you could try accepting the PR (and trigger the other machinery so that things get updated).
If you prefer that I keep the changes in a fork, I'm also happy to do so.

@pgrete pgrete changed the title Bump clang-format version to 10-13 Bump clang-format version to 10-12 Apr 28, 2022
@JoshuaSBrown
Copy link
Owner

@pgrete so sorry, I didn't see this, I get spammed with so many emails. I just now saw this issue, I'm guessing you simply created a fork.

@pgrete
Copy link
Author

pgrete commented Feb 2, 2023

@pgrete so sorry, I didn't see this, I get spammed with so many emails. I just now saw this issue, I'm guessing you simply created a fork.

I did and give that I had to update some other pieces in the CI infrastructure as well, the change to Parthenon was minimal.
Now that you brought this up, I'm thinking about moving the repo to the parthenon-hpc-lab organization so that there're more people in control.

@pgrete
Copy link
Author

pgrete commented Feb 2, 2023

Would you mind moving this repo @JoshuaSBrown (so that I could get rid of my fork) or should I move my fork?

@JoshuaSBrown
Copy link
Owner

@pgrete, if I cannot address this within a month I'll recommend you move the fork. ;)

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