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

touch file #1034

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from
Draft

touch file #1034

wants to merge 3 commits into from

Conversation

toxa81
Copy link
Collaborator

@toxa81 toxa81 commented Nov 28, 2024

No description provided.

@toxa81
Copy link
Collaborator Author

toxa81 commented Nov 29, 2024

@simonpintarelli @mtaillefumier I don't get this. Last merged PR to develop is greenlighted. All was good. I make a single character change, and this PR fails.

@toxa81
Copy link
Collaborator Author

toxa81 commented Nov 29, 2024

cscs-ci run default

@simonpintarelli
Copy link
Collaborator

I'm getting the same diff, clang-format removes the additional newline in src/_header_file_template.hpp

@toxa81
Copy link
Collaborator Author

toxa81 commented Nov 29, 2024

I'm getting the same diff, clang-format removes the additional newline in src/_header_file_template.hpp

yes, it's just to introduce a difference so CI can be triggered.

@simonpintarelli
Copy link
Collaborator

Ah ok, I was only looking at the formatting change.

I'll try to run it locally. The unit tests are failing with SIGPIPE.

@simonpintarelli
Copy link
Collaborator

ok, after undoing and reapplying the whitespace commit :), it looks like a glitch with sarus which caused the PIPE error.

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