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

Fixed eslint checks #3752

Merged
merged 1 commit into from
Aug 13, 2023
Merged

Conversation

criadoperez
Copy link
Contributor

Corrected format so eslint tests pass.

eslitnt actions was failing:

Run npx eslint --ext .php,.js .
  npx eslint --ext .php,.js .
  shell: /usr/bin/bash -e {0}

/home/runner/work/zoneminder/zoneminder/web/skins/classic/js/skin.js
  305:54  error  Statement inside of curly braces should be on next line                                                        brace-style
  305:55  error  Unexpected space(s) after '{'                                                                                  block-spacing
  305:70  error  Unexpected space(s) before '}'                                                                                 block-spacing
  305:71  error  Closing curly brace should be on the same line as opening curly brace or on the line after the previous block  brace-style

✖ 4 problems (4 errors, 0 warnings)
  4 errors and 0 warnings potentially fixable with the `--fix` option.

Error: Process completed with exit code 1.

@connortechnology connortechnology merged commit 07e60d9 into ZoneMinder:master Aug 13, 2023
15 of 19 checks passed
@criadoperez criadoperez deleted the fix/eslint branch August 17, 2023 23:52
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