{"action":"created","issue":{"url":"https://api.github.com/repos/ampproject/amphtml/issues/999","labels_url":"https://api.github.com/repos/ampproject/amphtml/issues/999/labels{/name}","comments_url":"https://api.github.com/repos/ampproject/amphtml/issues/999/comments","events_url":"https://api.github.com/repos/ampproject/amphtml/issues/999/events","html_url":"https://github.com/ampproject/amphtml/issues/999","id":118491036,"number":999,"title":"Remove the 'development' attribute mechanism for triggering validation","user":{"login":"Gregable","id":25519,"avatar_url":"https://avatars.githubusercontent.com/u/25519?v=3","gravatar_id":"","url":"https://api.github.com/users/Gregable","html_url":"https://github.com/Gregable","followers_url":"https://api.github.com/users/Gregable/followers","following_url":"https://api.github.com/users/Gregable/following{/other_user}","gists_url":"https://api.github.com/users/Gregable/gists{/gist_id}","starred_url":"https://api.github.com/users/Gregable/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Gregable/subscriptions","organizations_url":"https://api.github.com/users/Gregable/orgs","repos_url":"https://api.github.com/users/Gregable/repos","events_url":"https://api.github.com/users/Gregable/events{/privacy}","received_events_url":"https://api.github.com/users/Gregable/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-11-23T22:48:43Z","updated_at":"2015-11-23T22:51:31Z","closed_at":null,"body":"There are currently two ways of triggering AMP validation within the browser documented in the [AMP runtime section of the spec](https://github.com/ampproject/amphtml/blob/master/spec/amp-html-format.md#amp-runtime):\r\n\r\n1. Appending `#development=1` to the URL of the page.\r\n2. Adding the `development` attribute to the AMP runtime script tag, e.g. \r\n`<script src=\"https://cdn.ampproject.org/v0.js\" development></script>`.\r\n\r\nI'm considering removing the second option and looking for opinions.\r\n\r\nCurrently the attribute triggers validation which considers the development tag the one and only \"production warning\" reading `DEV_MODE_ENABLED development - please remove for prod`. This warning is emited to the javascript console but does not prevent the output from PASSing. This \"production warning\" seems to be a source of confusion. The idea is that we don't want validation to auto-fail due to triggering the validation itself, but we also don't want final published documents to contain this validation tag due to the performance hit. So, for development, we treat this as a warning, but in production, we treat this as an error.\r\n\r\nThe advantage of the development attribute is that it allows for CMS'es to automatically insert the tag when the developer views the page but remove it in user-facing contexts. However, now that the validator source is available, other options could be built for accomplishing the same thing. For example, the CMS could replace the AMP runtime script with a modified copy that always triggers validation, or validation could be performed server side."},"comment":{"url":"https://api.github.com/repos/ampproject/amphtml/issues/comments/159092965","html_url":"https://github.com/ampproject/amphtml/issues/999#issuecomment-159092965","issue_url":"https://api.github.com/repos/ampproject/amphtml/issues/999","id":159092965,"user":{"login":"dvoytenko","id":726049,"avatar_url":"https://avatars.githubusercontent.com/u/726049?v=3","gravatar_id":"","url":"https://api.github.com/users/dvoytenko","html_url":"https://github.com/dvoytenko","followers_url":"https://api.github.com/users/dvoytenko/followers","following_url":"https://api.github.com/users/dvoytenko/following{/other_user}","gists_url":"https://api.github.com/users/dvoytenko/gists{/gist_id}","starred_url":"https://api.github.com/users/dvoytenko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dvoytenko/subscriptions","organizations_url":"https://api.github.com/users/dvoytenko/orgs","repos_url":"https://api.github.com/users/dvoytenko/repos","events_url":"https://api.github.com/users/dvoytenko/events{/privacy}","received_events_url":"https://api.github.com/users/dvoytenko/received_events","type":"User","site_admin":false},"created_at":"2015-11-23T22:51:30Z","updated_at":"2015-11-23T22:51:30Z","body":"Yeah. With the ability to set `development=1` fragment, I also find `development` attribute redundant."}}