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

Fix the remaining cases of inconsistent spacing and trailing commas in objects, and enable the comma-dangle and object-curly-spacing ESLint rules #8478

Merged
merged 1 commit into from
Jun 4, 2017

Conversation

Snuffleupagus
Copy link
Collaborator

@Snuffleupagus Snuffleupagus commented Jun 2, 2017

http://eslint.org/docs/rules/comma-dangle
http://eslint.org/docs/rules/object-curly-spacing

Please note: This patch was created automatically, using the ESLint --fix command line option. In a couple of places this caused lines to become too long, and I've fixed those manually; please refer to the interdiff below for the only hand-edits in this patch.

diff --git a/gulpfile.js b/gulpfile.js
index d18b9c58..7d47fd8d 100644
--- a/gulpfile.js
+++ b/gulpfile.js
@@ -1247,7 +1247,8 @@ gulp.task('gh-pages-git', ['gh-pages-prepare', 'wintersmith'], function () {
   var reason = process.env['PDFJS_UPDATE_REASON'];

   safeSpawnSync('git', ['init'], { cwd: GH_PAGES_DIR, });
-  safeSpawnSync('git', ['remote', 'add', 'origin', REPO], { cwd: GH_PAGES_DIR, });
+  safeSpawnSync('git', ['remote', 'add', 'origin', REPO],
+                { cwd: GH_PAGES_DIR, });
   safeSpawnSync('git', ['add', '-A'], { cwd: GH_PAGES_DIR, });
   safeSpawnSync('git', [
     'commit', '-am', 'gh-pages site created via gulpfile.js script',

@timvandermeij
Copy link
Contributor

/botio-linux preview

@timvandermeij
Copy link
Contributor

/botio test

…n objects, and enable the `comma-dangle` and `object-curly-spacing` ESLint rules

http://eslint.org/docs/rules/comma-dangle
http://eslint.org/docs/rules/object-curly-spacing

*Please note:* This patch was created automatically, using the ESLint `--fix` command line option. In a couple of places this caused lines to become too long, and I've fixed those manually; please refer to the interdiff below for the only hand-edits in this patch.

```diff
diff --git a/gulpfile.js b/gulpfile.js
index d18b9c58..7d47fd8d 100644
--- a/gulpfile.js
+++ b/gulpfile.js
@@ -1247,7 +1247,8 @@ gulp.task('gh-pages-git', ['gh-pages-prepare', 'wintersmith'], function () {
   var reason = process.env['PDFJS_UPDATE_REASON'];

   safeSpawnSync('git', ['init'], { cwd: GH_PAGES_DIR, });
-  safeSpawnSync('git', ['remote', 'add', 'origin', REPO], { cwd: GH_PAGES_DIR, });
+  safeSpawnSync('git', ['remote', 'add', 'origin', REPO],
+                { cwd: GH_PAGES_DIR, });
   safeSpawnSync('git', ['add', '-A'], { cwd: GH_PAGES_DIR, });
   safeSpawnSync('git', [
     'commit', '-am', 'gh-pages site created via gulpfile.js script',
```
@Snuffleupagus
Copy link
Collaborator Author

@timvandermeij Thank you so much for reviewing and merging all the prior PRs that blocked this one! :-)

@Snuffleupagus
Copy link
Collaborator Author

/botio-linux lint

@pdfjsbot
Copy link

pdfjsbot commented Jun 3, 2017

From: Bot.io (Linux m4)


Received

Command cmd_lint from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.67.70.0:8877/66a22c858c6b14b/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Jun 3, 2017

From: Bot.io (Linux m4)


Success

Full output at http://54.67.70.0:8877/66a22c858c6b14b/output.txt

Total script time: 1.01 mins

  • Lint: Passed

@Snuffleupagus
Copy link
Collaborator Author

/botio test

@pdfjsbot
Copy link

pdfjsbot commented Jun 3, 2017

From: Bot.io (Linux m4)


Received

Command cmd_test from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.67.70.0:8877/79afd034502c2e9/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Jun 3, 2017

From: Bot.io (Windows)


Received

Command cmd_test from @Snuffleupagus received. Current queue size: 2

Live output at: http://54.215.176.217:8877/7eaa48283c4e7ea/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Jun 3, 2017

From: Bot.io (Linux m4)


Failed

Full output at http://54.67.70.0:8877/79afd034502c2e9/output.txt

Total script time: 17.02 mins

  • Font tests: Passed
  • Unit tests: Passed
  • Regression tests: FAILED

Image differences available at: http://54.67.70.0:8877/79afd034502c2e9/reftest-analyzer.html#web=eq.log

@pdfjsbot
Copy link

pdfjsbot commented Jun 3, 2017

From: Bot.io (Windows)


Success

Full output at http://54.215.176.217:8877/7eaa48283c4e7ea/output.txt

Total script time: 29.79 mins

  • Font tests: Passed
  • Unit tests: Passed
  • Regression tests: Passed

@timvandermeij
Copy link
Contributor

/botio-linux preview

@pdfjsbot
Copy link

pdfjsbot commented Jun 4, 2017

From: Bot.io (Linux m4)


Received

Command cmd_preview from @timvandermeij received. Current queue size: 0

Live output at: http://54.67.70.0:8877/4fdf7e34f9ab64d/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Jun 4, 2017

From: Bot.io (Linux m4)


Success

Full output at http://54.67.70.0:8877/4fdf7e34f9ab64d/output.txt

Total script time: 2.20 mins

Published

@timvandermeij
Copy link
Contributor

/botio-linux makeref

@pdfjsbot
Copy link

pdfjsbot commented Jun 4, 2017

From: Bot.io (Linux m4)


Received

Command cmd_makeref from @timvandermeij received. Current queue size: 0

Live output at: http://54.67.70.0:8877/abb1c1d4e65172f/output.txt

@timvandermeij timvandermeij merged commit 63b89c1 into mozilla:master Jun 4, 2017
@timvandermeij
Copy link
Contributor

Nice work on standardization! I'm happy that this is enforced using ESLint now instead of having to add this manually in reviews.

@pdfjsbot
Copy link

pdfjsbot commented Jun 4, 2017

From: Bot.io (Linux m4)


Success

Full output at http://54.67.70.0:8877/abb1c1d4e65172f/output.txt

Total script time: 16.07 mins

  • Lint: Passed
  • Make references: Passed
  • Check references: Passed

@Snuffleupagus Snuffleupagus deleted the ESLint-object-styles branch June 4, 2017 18:23
movsb pushed a commit to movsb/pdf.js that referenced this pull request Jul 14, 2018
Fix the remaining cases of inconsistent spacing and trailing commas in objects, and enable the `comma-dangle` and `object-curly-spacing` ESLint rules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants