From d910a77ddfb1b2a61b51d4412529c78b6837f7bd Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Wed, 15 Sep 2021 22:07:43 +0200 Subject: [PATCH] Update example `class` syntax to account for Prettier-formatting --- Style-Guide.md | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/Style-Guide.md b/Style-Guide.md index 9d99352..9aaf29d 100644 --- a/Style-Guide.md +++ b/Style-Guide.md @@ -1,6 +1,6 @@ This page outlines the style conventions that PDF.js follows to maintain a consistent code base. We ask each contributor that creates a pull request to adhere to these conventions. -Most of these conventions are checked automatically by a linter (Prettier) after each push to a branch of a pull request. Refer to https://github.com/mozilla/pdf.js/wiki/Contributing#4-run-lint-and-testing for information about running the linter locally. +Most of these conventions are checked automatically by a linter (ESLint/Prettier) after each push to a branch of a pull request. Refer to https://github.com/mozilla/pdf.js/wiki/Contributing#4-run-lint-and-testing for information about running the linter locally. ## General * Indentation: 2 spaces @@ -57,8 +57,11 @@ class ClassName { ... } - aVeryVeryVeryVeryVeryLongFunctionName - (arg1, arg2, ...) { + aVeryVeryVeryVeryVeryLongFunctionName( + arg1, + arg2, + ... + ) { ... } } @@ -77,8 +80,11 @@ var ClassName = (function ClassNameClosure() { ... }, - aVeryVeryVeryVeryVeryLongFunctionName - (arg1, arg2, ...) { + aVeryVeryVeryVeryVeryLongFunctionName( + arg1, + arg2, + ... + ) { ... }, };