From 8ed376e6d428bb049d417e0d2baee961b4237413 Mon Sep 17 00:00:00 2001 From: Duy Khoa Date: Tue, 2 Jan 2018 11:50:53 +0800 Subject: [PATCH 1/2] Remove trailing spaces and extra lines in CONTRIBUTING.md --- CONTRIBUTING.md | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index ee1615a854..597f6b06a4 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -42,7 +42,7 @@ If you find an issue, please do file it on the repository. The [Polymer/polymer For issues with elements the team maintains, please file directly on the element's repository. If you're not sure if a bug stems from the element or the library, air toward filing it on the element and we'll move the issue if necessary. -Please file issues using the issue template provided, filling out as many fields as possible. We love examples for addressing issues - issues with a Plunkr, jsFiddle, or jsBin will be much easier for us to work on quickly. You can start with [this jsbin](http://jsbin.com/luhaxab/edit) which sets up the basics to demonstrate a Polymer element. +Please file issues using the issue template provided, filling out as many fields as possible. We love examples for addressing issues - issues with a Plunkr, jsFiddle, or jsBin will be much easier for us to work on quickly. You can start with [this jsbin](http://jsbin.com/luhaxab/edit) which sets up the basics to demonstrate a Polymer element. Occasionally we'll close issues if they appear stale or are too vague - please don't take this personally! Please feel free to re-open issues we've closed if there's something we've missed and they still need to be addressed. @@ -122,14 +122,13 @@ Navigate to: ### Running Polymer element unit tests -To run the element unit tests, you need a global -install of `web-component-tester` or `polyserve` (or both). +To run the element unit tests, you need a global install of `web-component-tester` or `polyserve` (or both). 1. Clone the element repo. 2. Install the dependencies. - bower install + bower install 3. Run the tests: @@ -148,7 +147,5 @@ install of `web-component-tester` or `polyserve` (or both). By default, `web-component-tester` runs tests on all installed browsers. You can configure it to run tests on a subset of available browsers, or to run tests remotely using Sauce Labs. -See the [`web-component-tester` README](https://github.com/Polymer/web-component-tester) for -information on configuring the tool. - - +See the [`web-component-tester` README](https://github.com/Polymer/web-component-tester) for +information on configuring the tool. From 6f7b860899269092f011f0d8823b560803836670 Mon Sep 17 00:00:00 2001 From: Duy Khoa Date: Thu, 4 Jan 2018 15:22:16 +0800 Subject: [PATCH 2/2] Use double tabs --- CONTRIBUTING.md | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 597f6b06a4..6aff080d61 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -90,15 +90,15 @@ To run the Polymer library unit tests: 2. Install the dependencies: - npm install && bower install + npm install && bower install 3. Run the tests: - npm test + npm test Or if you have `web-component-tester` installed locally: - wct + wct To run individual test suites: @@ -110,11 +110,11 @@ Or: For example: - npm test test/unit/template.html + npm test test/unit/template.html You can also run tests in the browser: - polyserve + polyserve Navigate to: @@ -124,23 +124,23 @@ Navigate to: To run the element unit tests, you need a global install of `web-component-tester` or `polyserve` (or both). -1. Clone the element repo. +1. Clone the element repo. -2. Install the dependencies. +1. Install the dependencies. - bower install + bower install -3. Run the tests: +1. Run the tests: - wct + wct - Or run the tests in a browser: + Or run the tests in a browser: - polyserve + polyserve - Navigate to: + Navigate to: - http://localhost:8080/components/element-name/test/runner.html + http://localhost:8080/components/element-name/test/runner.html ### Configuring `web-component-tester`