Skip to content

tidy: Avoid contamination of headers#986

Open
KSkwarczynski wants to merge 1 commit into
developfrom
kskwarczynski/tidy/NoContamination
Open

tidy: Avoid contamination of headers#986
KSkwarczynski wants to merge 1 commit into
developfrom
kskwarczynski/tidy/NoContamination

Conversation

@KSkwarczynski

Copy link
Copy Markdown
Member

Pull request description

We use pragma diagnostic to suppress warnings for a give code line or whole file.
Adding them to header file without start/end means we contaminate large part of codebase.

This is .py file so not huge problem, but fixing nevertheless.


@github-actions

Copy link
Copy Markdown

Hi @KSkwarczynski, thank you for contributing to MaCh3! Please wait for MaCh3 developers to review your PR. If no one answers within a week, please message people from this list: https://github.com/orgs/mach3-software/teams/mach3admin . While waiting, please enjoy this

meme

Use this action on your projects. Use jokes on issues instead.

@github-actions

Copy link
Copy Markdown

Super-linter summary

Language Validation result
YAML Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

@github-actions

Copy link
Copy Markdown

Hi, I am MaCh3-Telemetry bot

Memory and CPU Usage Summary

### Memory and CPU Usage Summary

| Metric  |     Mean    |      Max    |
|---------|-------------|-------------|
| **RAM** |   359.72 MB |   369.56 MB |
| **CPU** |    97.01 %  |   106.00 %  |

Memory Usage Plot

You can view the memory usage plot directly in the job summary from the link above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant