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.

Sign up for GitHub

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

Jump to bottom

Clang-format workflow is broken #12667

Closed
cgcgcg opened this issue Jan 15, 2024 · 2 comments · Fixed by #12670
Closed

Clang-format workflow is broken #12667

cgcgcg opened this issue Jan 15, 2024 · 2 comments · Fixed by #12670
Assignees
@csiefer2 @GrahamBenHarper
Labels
type: bug The primary issue is a bug in Trilinos code or tests

Comments

@cgcgcg
Copy link
Contributor

cgcgcg commented Jan 15, 2024

Clang format seems to be happy with my PR, but it fails to post a message. This might be yet another case of bad permissions:
https://github.com/trilinos/Trilinos/actions/runs/7533991464/job/20507564609
@cgcgcg cgcgcg added the type: bug The primary issue is a bug in Trilinos code or tests label Jan 15, 2024
@cgcgcg cgcgcg linked a pull request Jan 16, 2024 that will close this issue
Attempt to fix clang-format workflow #12670
Merged
@cgcgcg
Copy link
Contributor Author

cgcgcg commented Jan 16, 2024

Also affected PR #12415.

@cgcgcg
Copy link
Contributor Author

cgcgcg commented Jan 19, 2024

Fixed by #12670

@cgcgcg cgcgcg closed this as completed Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@csiefer2 csiefer2

@GrahamBenHarper GrahamBenHarper

Labels
type: bug The primary issue is a bug in Trilinos code or tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Attempt to fix clang-format workflow cgcgcg/Trilinos
3 participants
@cgcgcg @csiefer2 @GrahamBenHarper