jorgebay commented on a change in pull request #1539:
URL: https://github.com/apache/tinkerpop/pull/1539#discussion_r818437551
##########
File path:
gremlin-javascript/src/main/javascript/gremlin-javascript/lib/driver/client.js
##########
@@ -46,13 +45,13 @@ class Client {
*/
constructor(url, options = {}) {
this._options = options;
- if (this._options.processor === 'session') {
+ if (this._options.processor === "session") {
Review comment:
We don't have a linter for the project but we should follow the rest of
the other files as conventions.
There are several line changes related to strings and overriding existing
formatting, it's likely that it's your IDE but in any case, we should avoid
them to reduce history noise.
In general, you can check out airbnb's style guide:
https://github.com/airbnb/javascript#strings
Would it be possible to revert the unnecessary style changes?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
