Skip to content

Fix civilint whitespace errors#389

Merged
totten merged 2 commits intototten:masterfrom
colemanw:fixwhitespace
Jan 27, 2025
Merged

Fix civilint whitespace errors#389
totten merged 2 commits intototten:masterfrom
colemanw:fixwhitespace

Conversation

@colemanw
Copy link
Copy Markdown
Contributor

@colemanw colemanw commented Jan 27, 2025

Before

FILE: ...alone-clean/web/core/ext/user_dashboard/user_dashboard.civix.php
----------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
----------------------------------------------------------------------
 76 | ERROR | [x] Expected 1 blank line after function; 2 found
 79 | ERROR | [x] The closing brace for the class must have an empty
    |       |     line before it
----------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

And

FILE: ...build-2/web/src/ext/oauth-client/CRM/OAuth/DAO/OAuthSysToken.php
----------------------------------------------------------------------
FOUND 15 ERRORS AFFECTING 15 LINES
----------------------------------------------------------------------
 11 | ERROR | [x] Whitespace found at end of line
 12 | ERROR | [x] Whitespace found at end of line
 13 | ERROR | [x] Whitespace found at end of line
 14 | ERROR | [x] Whitespace found at end of line
 15 | ERROR | [x] Whitespace found at end of line
 16 | ERROR | [x] Whitespace found at end of line
 17 | ERROR | [x] Whitespace found at end of line
 18 | ERROR | [x] Whitespace found at end of line
 19 | ERROR | [x] Whitespace found at end of line
 20 | ERROR | [x] Whitespace found at end of line
 21 | ERROR | [x] Whitespace found at end of line
 22 | ERROR | [x] Whitespace found at end of line
 23 | ERROR | [x] Whitespace found at end of line
 24 | ERROR | [x] Whitespace found at end of line
 25 | ERROR | [x] Whitespace found at end of line
----------------------------------------------------------------------
PHPCBF CAN FIX THE 15 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

After

CIviLint is happy.

@colemanw colemanw changed the title Fix civilint whitespace error in civix.php file Fix civilint whitespace errors Jan 27, 2025
@totten
Copy link
Copy Markdown
Owner

totten commented Jan 27, 2025

Works for me. 👍

@totten totten merged commit a1f9acf into totten:master Jan 27, 2025
@colemanw colemanw deleted the fixwhitespace branch January 28, 2025 01:18
@colemanw
Copy link
Copy Markdown
Contributor Author

Thanks @totten

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