Fix phpcs errors and warnings

Hello @davidwbarratt,
A few Phpcs issues are still there, so I think it'll be a good idea to reopen this issue only to address them, instead of creating new issue for the same purpose.
As of now, I'm getting all these Phpcs issues which can be fixed:

FILE: /app/web/themes/custom/sailor/js/script.js
----------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
----------------------------------------------------------------------
 15 | ERROR | [x] Expected 1 space after FUNCTION keyword; 0 found
 18 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected
    |       |     "FALSE" but found "false"
 19 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected
    |       |     "FALSE" but found "false"
----------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------


FILE: /app/web/themes/custom/sailor/sailor.theme
----------------------------------------------------------------------
FOUND 10 ERRORS AFFECTING 10 LINES
----------------------------------------------------------------------
  12 | ERROR | [x] Use statements should be sorted alphabetically.
     |       |     The first wrong one is
     |       |     Drupal\Component\Utility\Html.
 272 | ERROR | [x] Functions must not contain multiple empty lines in
     |       |     a row; found 2 empty lines
 275 | ERROR | [x] Line indented incorrectly; expected 2 spaces,
     |       |     found 1
 279 | ERROR | [x] Closing brace indented incorrectly; expected 1
     |       |     spaces, found 2
 292 | ERROR | [x] Functions must not contain multiple empty lines in
     |       |     a row; found 2 empty lines
 302 | ERROR | [x] Whitespace found at end of line
 306 | ERROR | [x] Whitespace found at end of line
 312 | ERROR | [x] Whitespace found at end of line
 588 | ERROR | [x] Inline comments must end in full-stops,
     |       |     exclamation marks, question marks, colons, or
     |       |     closing parentheses
 910 | ERROR | [x] Use null coalesce operator instead of ternary
     |       |     operator.
----------------------------------------------------------------------
PHPCBF CAN FIX THE 10 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------


FILE: /app/web/themes/custom/sailor/webpack.config.js
----------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
----------------------------------------------------------------------
  34 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected
     |       |     "TRUE" but found "true"
 102 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected
     |       |     "TRUE" but found "true"
----------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

Time: 322ms; Memory: 18MB

I have fixed phpcs Issues could you please review it.

Thankyou !

Hi,

I have verified the PHPCS issue with the #3320722 merge request applied cleanly in the Drupal 10 version.

Testing Steps:
-> Install the Drupal 10 version.
-> Open the command line and run PHPCS on this module.
-> Check for errors and warnings.

Testing Results:
Showing no errors and warnings.

Attached The after screenshots. Based on these results, the issue can be moved to RTBC.

RTBC+1

Thanks all for the prompt action. A small correction on the issue metadata, which I forgot to change previously. The phpcs issues seemed to be persistent on the latest version of the theme as well, though the MR solves all the Phpcs issues on the latest version as well!

Automatically closed - issue fixed for 2 weeks with no activity.