Merge pull request #10 from prebid/master

Merge prebid/Prebid into adform/Prebid

.editorconfig

@@ -0,0 +1,11 @@
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = true
+
+[*.js]
+charset = utf-8
+indent_style = space
+indent_size = 2
+trim_trailing_whitespace = true

.eslintrc

@@ -0,0 +1,42 @@
+{
+  "env": {
+    "browser": true,
+    "commonjs": true
+  },
+  "extends": "standard",
+  "globals": {
+    "$$PREBID_GLOBAL$$": false
+  },
+  "parserOptions": {
+    "sourceType": "module"
+  },
+  "rules": {
+    "comma-dangle": "off",
+    "semi": "off",
+    "space-before-function-paren": "off",
+
+    // Exceptions below this line are temporary, so that eslint can be added into the CI process.
+    // Violations of these styles should be fixed, and the exceptions removed over time.
+    //
+    // See Issue #1111.
+    "brace-style": "off",
+    "camelcase": "off",
+    "eqeqeq": "off",
+    "import/first": "off",
+    "no-control-regex": "off",
+    "no-mixed-operators": "off",
+    "no-multiple-empty-lines": "off",
+    "no-redeclare": "off",
+    "no-return-assign": "off",
+    "no-throw-literal": "off",
+    "no-undef": "off",
+    "no-unused-vars": "off",
+    "no-use-before-define": "off",
+    "no-useless-call": "off",
+    "no-useless-escape": "off",
+    "one-var": "off",
+    "standard/no-callback-literal": "off",
+    "standard/object-curly-even-spacing": "off",
+    "valid-typeof": "off"
+  }
+}

.github/PULL_REQUEST_TEMPLATE.md

@@ -26,6 +26,9 @@ Thank you for your pull request. Please make sure this PR is scoped to one chang
   }
 }
 ```
+
+Be sure to test the integration with your adserver using the [Hello World](/integrationExamples/gpt/hello_world.html) sample page.
+
 - contact email of the adapter’s maintainer
 - [ ] official adapter submission
 

CONTRIBUTING.md

@@ -1,24 +1,42 @@
 # Contributing to Prebid.js
-Contributions are always welcome. To contribute, [fork](https://help.github.com/articles/fork-a-repo/) Prebid.js, commit your changes, and [open a pull request](https://help.github.com/articles/using-pull-requests/).
+Contributions are always welcome. To contribute, [fork](https://help.github.com/articles/fork-a-repo/) Prebid.js,
+commit your changes, and [open a pull request](https://help.github.com/articles/using-pull-requests/) against the
+master branch.
 
-## Pull Requests
-Please make sure that pull requests are scoped to one change, and that any added or changed code includes tests with greater than 80% code coverage. See [Testing Prebid.js](#testing-prebidjs) for help on writing tests.
+Pull requests must have 80% code coverage before beign considered for merge.
+Additional details about the process can be found [here](./pr_review.md).
 
 ## Issues
-[prebid.org](http://prebid.org/) contains documentation that may help answer questions you have about using Prebid.js. If you can't find the answer there, try searching for a similar issue on the [issues page](https://github.com/prebid/Prebid.js/issues). If you don't find an answer there, [open a new issue](https://github.com/prebid/Prebid.js/issues/new).
+[prebid.org](http://prebid.org/) contains documentation that may help answer questions you have about using Prebid.js.
+If you can't find the answer there, try searching for a similar issue on the [issues page](https://github.com/prebid/Prebid.js/issues).
+If you don't find an answer there, [open a new issue](https://github.com/prebid/Prebid.js/issues/new).
 
 ## Documentation
 If you have a documentation issue or pull request, please open a ticket or PR in the [documentation repository](https://github.com/prebid/prebid.github.io).
 
-## Testing Prebid.js
-Pull requests to the Prebid.js library will need to include tests with greater than 80% code coverage for any changed/added code before they can be merged into master.
+## Writing Tests
 
-This section describes how to test code in the Prebid.js repository to help prepare your pull request.
+Prebid uses [Mocha](http://mochajs.org/) and [Chai](http://chaijs.com/) for unit tests. [Sinon](http://sinonjs.org/)
+provides mocks, stubs, and spies. [Karma](https://karma-runner.github.io/1.0/index.html) runs the tests and generates
+code coverage reports at `build/coverage/lcov/lcov-report/index.html`.
 
-### Writing tests
+Tests are stored in the [test/spec](test/spec) directory. Tests for Adapters are located in [test/spec/adapters](test/spec/adapters).
+They can be run with the following commands:
+
+- `gulp test` - run the test suite once (`npm test` is aliased to call `gulp test`)
+- `gulp serve` - run the test suite once, but re-run it whenever a file in the `src` or `test` directory is modified
+
+Before a Pull Request will be considered for merge:
+
+- All new and existing tests must pass
+- Added or modified code must have greater than 80% coverage
+
+If you are submitting an adapter, you can also use the [Hello World](integrationExamples/gpt/hello_world.html) example page to test integration with your server.
+
+### Test Guidelines
 When you are adding code to Prebid.js, or modifying code that isn't covered by an existing test, test the code according to these guidelines:
 
-- If the module you are working on is already partially tested by a file within the `test` directory, add tests to that file
+- If the module you are working on is already partially tested by a file within the `test/spec` directory, add tests to that file
 - If the module does not have any tests, create a new test file
 - Group tests in a `describe` block
 - Test individual units of code within an `it` block
@@ -36,31 +54,8 @@ When you are adding code to Prebid.js, or modifying code that isn't covered by a
 - If you need to check `adloader.loadScript` in a test, use a `stub` rather than a `spy`. `spy`s trigger a network call which can result in a `script error` and cause unrelated unit tests to fail. `stub`s will let you gather information about the `adloader.loadScript` call without affecting external resources
 - When writing tests you may use ES2015 syntax if desired
 
-### Running tests
-After checking out the Prebid.js repository and installing dev dependencies with `npm install`, use the following commands to run tests as you are working on code:
-
-- `gulp test` will run the test suite once (`npm test` is aliased to call `gulp test`)
-- `gulp serve` will run tests once and stay open, re-running tests whenever a file in the `src` or `test` directory is modified
-
-### Checking results and code coverage
-Check the test results using these guidelines:
-
-- Look at the total number of tests run, passed, and failed in the shell window.
-- If all tests are passing, great.
-- Otherwise look for errors printed in the console for a description of the failing test.
-- You may need to iterate on your code or tests until all tests are passing.
-- Make sure existing tests still pass.
-- There is a table below the testing report that shows code coverage percentage, for each file under the `src` directory.
-- Each time you run tests, a code coverage report is generated in `build/coverage/lcov/lcov-report/index.html`.
-- This is a static HTML page that you can load in your browser.
-- On that page, navigate to the file you are testing to see which lines are being tested.
-- Red indicates that a line isn't covered by a test.
-- Gray indicates a line that doesn't need coverage, such as a comment or blank line.
-- Green indicates a line that is covered by tests.
-- The code you have added or modified must have greater than 80% coverage to be accepted.
-
-### Examples
-Prebid.js already has lots of tests. Read them to see how Prebid.js is tested, and for inspiration:
+### Test Examples
+Prebid.js already has many tests. Read them to see how Prebid.js is tested, and for inspiration:
 
 - Look in `test/spec` and its subdirectories
 - Tests for bidder adaptors are located in `test/spec/adapters`
@@ -82,10 +77,3 @@ describe('<Adapter>', () => {
   // Add other `describe` or `it` blocks as necessary
 });
 ```
-
-### Resources
-The Prebid.js testing stack contains some of the following tools. It may be helpful to consult their documentation during the testing process.
-
-- [Mocha - test framework](http://mochajs.org/)
-- [Chai - BDD/TDD assertion library](http://chaijs.com/)
-- [Sinon - spy, stub, and mock library](http://sinonjs.org/)

README.md

@@ -9,7 +9,9 @@
 
 > A free and open source library for publishers to quickly implement header bidding.
 
-This README is for developers who want to contribute to Prebid.js.  For user-facing documentation, see [Prebid.org](http://prebid.org).
+This README is for developers who want to contribute to Prebid.js.
+Additional documentation can be found at [the Prebid homepage](http://prebid.org).
+Working examples can be found in [the developer docs](http://prebid.org/dev-docs/getting-started.html).
 
 **Table of Contents**
 
@@ -26,13 +28,13 @@ This README is for developers who want to contribute to Prebid.js.  For user-fac
     $ cd Prebid.js
     $ yarn install
 
-Prebid now supports the `yarn` npm client. This is an alternative to using `npm` for package management, though `npm` will continue to work as before.
+Prebid also supports the `yarn` npm client. This is an alternative to using `npm` for package management, though `npm` will continue to work as before.
 
-For more info about yarn see https://yarnpkg.com
+For more info, see [the Yarn documentation](https://yarnpkg.com).
 
 <a name="Build"></a>
 
-## Build for Dev
+## Build for Development
 
 To build the project on your local machine, run:
 
@@ -148,6 +150,8 @@ A watch is also in place that will run continuous tests in the terminal as you e
 
 Many SSPs, bidders, and publishers have contributed to this project. [60+ Bidders](https://github.com/prebid/Prebid.js/tree/master/src/adapters) are supported by Prebid.js.
 
+For guidelines, see [Contributing](./CONTRIBUTING.md).
+
 Our PR review process can be found [here](https://github.com/prebid/Prebid.js/tree/master/pr_review.md).
 
 ### Add a Bidder Adapter
@@ -158,9 +162,9 @@ Please **do NOT load Prebid.js inside your adapter**. If you do this, we will re
 
 ### Code Quality
 
-Code quality is defined by `.jscs` and `.jshint` files and errors are reported in the terminal.
+Code quality is defined by `.eslintrc` and errors are reported in the terminal.
 
-If you are contributing code, you should configure your editor with the provided `.jscs` and `.jshint` settings.
+If you are contributing code, you should [configure your editor](http://eslint.org/docs/user-guide/integrations#editors) with the provided `.eslintrc` settings.
 
 ### Unit Testing with Karma
 

adapters.json

@@ -7,6 +7,7 @@
     "adform",
     "adkernel",
     "admedia",
+    "adyoulike",
     "bidfluence",
     "vertamedia",
     "aol",
@@ -30,8 +31,10 @@
     "openx",
     "piximedia",
     "pubmatic",
+    "pubgears",
     "pulsepoint",
     "pulsepointLite",
+    "quantcast",
     "rhythmone",
     "rubicon",
     "smartyads",
@@ -64,6 +67,8 @@
     "atomx",
     "tapsense",
     "trion",
+    "prebidServer",
+    "adsupply",
     {
       "appnexus": {
         "alias": "brealtime"
@@ -86,9 +91,10 @@
     },
     {
       "appnexusAst": {
-        "supportedMediaTypes": ["video"]
+        "supportedMediaTypes": ["video", "native"]
       }
-    },    {
+    },
+    {
       "vertamedia": {
         "supportedMediaTypes": ["video"]
       }
@@ -138,5 +144,15 @@
       "rhythmone": {
         "supportedMediaTypes": ["video"]
       }
+    },
+    {
+      "admixer": {
+              "supportedMediaTypes": ["video"]
+      }
+    },
+    {
+      "conversant": {
+        "supportedMediaTypes": ["video"]
+      }
     }
 ]

analytics.json

@@ -1 +1 @@
-[]
+[]

browsers.json

@@ -95,14 +95,6 @@
     "device": null,
     "os": "Windows"
   },
-  "bs_ie_9_windows_7": {
-    "base": "BrowserStack",
-    "os_version": "7",
-    "browser": "ie",
-    "browser_version": "9.0",
-    "device": null,
-    "os": "Windows"
-  },
   "bs_firefox_46_windows_7": {
     "base": "BrowserStack",
     "os_version": "7",