Skip to content

Fix opal_progress.h copyright header. #8910

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

Conversation

awlauria
Copy link
Contributor

@awlauria awlauria commented May 3, 2021

Fix regression from the opal formatting changes
(4c4d520) that put the copyright headers into a bad state.

Fixes #8868

Signed-off-by: Austen Lauria [email protected]

@awlauria awlauria requested review from hjelmn and devreal May 3, 2021 14:05
@awlauria
Copy link
Contributor Author

awlauria commented May 3, 2021

@devreal I didn't see any other headers in a bad state from the opal changes.

Fix regression from the opal formatting changes
(4c4d520) that put the copyright headers into a bad state.

Signed-off-by: Austen Lauria <[email protected]>
@awlauria awlauria force-pushed the fix_opal_progress_h_copyright_formatting_master branch from 4eca9d2 to 00d61d0 Compare May 3, 2021 14:18
Copy link
Contributor

@devreal devreal left a comment

Choose a reason for hiding this comment

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

LGTM

@awlauria awlauria merged commit e8857f0 into open-mpi:master May 3, 2021
@awlauria awlauria deleted the fix_opal_progress_h_copyright_formatting_master branch May 3, 2021 15:24
@hjelmn
Copy link
Member

hjelmn commented May 3, 2021

Wow. That formatting is just plain bizarre. Will take a look and see if a big needs to be filed with llvm.

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.

Formatting changes have mangled some copyright headers
3 participants