{"action":"opened","issue":{"url":"https://api.github.com/repos/openlayers/ol3/issues/3519","labels_url":"https://api.github.com/repos/openlayers/ol3/issues/3519/labels{/name}","comments_url":"https://api.github.com/repos/openlayers/ol3/issues/3519/comments","events_url":"https://api.github.com/repos/openlayers/ol3/issues/3519/events","html_url":"https://github.com/openlayers/ol3/issues/3519","id":67080195,"number":3519,"title":"Enhance CONTRIBUTING.md","user":{"login":"bartvde","id":319678,"avatar_url":"https://avatars.githubusercontent.com/u/319678?v=3","gravatar_id":"","url":"https://api.github.com/users/bartvde","html_url":"https://github.com/bartvde","followers_url":"https://api.github.com/users/bartvde/followers","following_url":"https://api.github.com/users/bartvde/following{/other_user}","gists_url":"https://api.github.com/users/bartvde/gists{/gist_id}","starred_url":"https://api.github.com/users/bartvde/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bartvde/subscriptions","organizations_url":"https://api.github.com/users/bartvde/orgs","repos_url":"https://api.github.com/users/bartvde/repos","events_url":"https://api.github.com/users/bartvde/events{/privacy}","received_events_url":"https://api.github.com/users/bartvde/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-04-08T08:47:35Z","updated_at":"2015-04-08T08:47:35Z","closed_at":null,"body":"With:\r\n\r\n1) at the sprint we added assertion messages for all assertions, so please when you add a new assertion, also add the message for those\r\n2) any method with the @api annotation must have a good one sentence plain text description. Additional detail is great if it adds value. It's also nice to use full sentences.\r\n3) all examples (also the ones in existing pull requests) must use the new templating system for details see: https://github.com/openlayers/ol3/blob/master/examples_src/readme.md\r\n4) pull requests now show increase or decrease in test coverage, if test coverage drops below an overall threshold (which we have currently set at 65%), the PR will fail. It would be good wherever possible to increase test coverage with your PR of course, and the output from istanbul / coveralls can provide useful insights into what is tested (which might be different than what you expect for some cases). To run locally use npm install followed by npm run test-coverage (or ./build.py test-coverage) and check the HTML output in the coverage directory with a browser."}}