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

Core: fix code style #2422

Merged
merged 1 commit into from Apr 12, 2022
Merged

Core: fix code style #2422

merged 1 commit into from Apr 12, 2022

Conversation

bytestream
Copy link
Member

@bytestream bytestream commented Apr 12, 2022

Description

Build is failing as a result of #2154

Running "jscs:all" (jscs) task
If statement without curly braces at src/core.js :
  1033 |  // as a literal part of a name/id or any selector.
  1034 |  escapeCssMeta: function( string ) {
  1035 |   if (string === undefined) return "";
-----------^
  1036 |   return string.replace( /([\\!"#$%&'()*+,./:;<=>?@\[\]^`{|}~])/g, "\\$1" );
  1037 |  },
Missing space after opening round bracket at src/core.js :
  1033 |  // as a literal part of a name/id or any selector.
  1034 |  escapeCssMeta: function( string ) {
  1035 |   if (string === undefined) return "";
---------------^
  1036 |   return string.replace( /([\\!"#$%&'()*+,./:;<=>?@\[\]^`{|}~])/g, "\\$1" );
  1037 |  },
Missing space before closing round bracket at src/core.js :
  1033 |  // as a literal part of a name/id or any selector.
  1034 |  escapeCssMeta: function( string ) {
  1035 |   if (string === undefined) return "";
-----------------------------------^
  1036 |   return string.replace( /([\\!"#$%&'()*+,./:;<=>?@\[\]^`{|}~])/g, "\\$1" );
  1037 |  },
>> 3 code style errors found!
Warning: Task "jscs:all" failed. Use --force to continue.

Aborted due to warnings.
npm ERR! code ELIFECYCLE
npm ERR! errno 3
npm ERR! jquery-validation@1.19.4-pre prepublish: `grunt`
npm ERR! Exit status 3
npm ERR! 
npm ERR! Failed at the jquery-validation@1.19.4-pre prepublish script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

Thank you!

@staabm staabm merged commit 900a90b into jquery-validation:master Apr 12, 2022
@bytestream bytestream deleted the codestyle branch Apr 12, 2022
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.

None yet

2 participants