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

Add a GitHub workflow to check for whitespace issues #1242

Merged
merged 1 commit into from
May 3, 2024

Conversation

ac000
Copy link
Member

@ac000 ac000 commented May 2, 2024

If it fails you can check the 'git log --check' output of the workflow to see what the issue is. E.g

--- 93ec013 Oops...
README.md:1: trailing whitespace.
+# NGINX Unit

If it fails you can check the 'git log --check' output of the workflow
to see what the issue is. E.g

  --- 93ec013 Oops...
  README.md:1: trailing whitespace.
  +# NGINX Unit

Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
@ac000 ac000 requested a review from callahad May 2, 2024 13:29
Copy link
Collaborator

@callahad callahad left a comment

Choose a reason for hiding this comment

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

lgtm

@ac000 ac000 merged commit ff2e0f4 into nginx:master May 3, 2024
19 checks passed
@ac000 ac000 deleted the wf-ws branch May 3, 2024 01:34
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