Skip to content

ci: add whitespace checks - #194

Merged
andybavier merged 3 commits into
mainfrom
ci/whitespace-check
Aug 7, 2026
Merged

ci: add whitespace checks#194
andybavier merged 3 commits into
mainfrom
ci/whitespace-check

Conversation

@andybavier

@andybavier andybavier commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add pre-commit hooks for trailing whitespace and end-of-file normalization
  • run the hooks in GitHub Actions CI
  • preserve protocol-specific packet-capture fixture endings

Validation

  • YAML parsing
  • git diff --check

@andybavier
andybavier requested a review from a team August 7, 2026 18:41
@andybavier
ci: add whitespace checks
a9df49c 
Signed-off-by: Andy Bavier <andybavier@gmail.com>
@andybavier
ci: preserve packet capture fixture endings
8b57779 
Signed-off-by: Andy Bavier <andybavier@gmail.com>
@andybavier
andybavier force-pushed the ci/whitespace-check branch from cc5c049 to 8b57779 Compare August 7, 2026 18:42
@andybavier
ci: add license header to whitespace config
4f16128 
Signed-off-by: Andy Bavier <andybavier@gmail.com>
@andybavier
andybavier merged commit 6e914c3 into main Aug 7, 2026
14 checks passed
@andybavier
andybavier deleted the ci/whitespace-check branch August 7, 2026 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cmaciocco cmaciocco cmaciocco approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@andybavier @cmaciocco