{"action":"created","issue":{"url":"https://api.github.com/repos/FreeCodeCamp/FreeCodeCamp/issues/5880","labels_url":"https://api.github.com/repos/FreeCodeCamp/FreeCodeCamp/issues/5880/labels{/name}","comments_url":"https://api.github.com/repos/FreeCodeCamp/FreeCodeCamp/issues/5880/comments","events_url":"https://api.github.com/repos/FreeCodeCamp/FreeCodeCamp/issues/5880/events","html_url":"https://github.com/FreeCodeCamp/FreeCodeCamp/pull/5880","id":124892377,"number":5880,"title":"Improved regex to properly test \"Do not change code below this line\"","user":{"login":"ShaggyTech","id":6378521,"avatar_url":"https://avatars.githubusercontent.com/u/6378521?v=3","gravatar_id":"","url":"https://api.github.com/users/ShaggyTech","html_url":"https://github.com/ShaggyTech","followers_url":"https://api.github.com/users/ShaggyTech/followers","following_url":"https://api.github.com/users/ShaggyTech/following{/other_user}","gists_url":"https://api.github.com/users/ShaggyTech/gists{/gist_id}","starred_url":"https://api.github.com/users/ShaggyTech/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ShaggyTech/subscriptions","organizations_url":"https://api.github.com/users/ShaggyTech/orgs","repos_url":"https://api.github.com/users/ShaggyTech/repos","events_url":"https://api.github.com/users/ShaggyTech/events{/privacy}","received_events_url":"https://api.github.com/users/ShaggyTech/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-01-05T04:12:47Z","updated_at":"2016-01-05T05:49:44Z","closed_at":"2016-01-05T05:49:44Z","pull_request":{"url":"https://api.github.com/repos/FreeCodeCamp/FreeCodeCamp/pulls/5880","html_url":"https://github.com/FreeCodeCamp/FreeCodeCamp/pull/5880","diff_url":"https://github.com/FreeCodeCamp/FreeCodeCamp/pull/5880.diff","patch_url":"https://github.com/FreeCodeCamp/FreeCodeCamp/pull/5880.patch"},"body":"The last test will now properly check to make sure the camper did not change any of the code \"below the line\".\r\n\r\nIt also allows for spaces and newlines as long as they are not placed between the text/code below the line.\r\n\r\nFixes #5875\r\n\r\nThis will also work for code that should not be changed in other waypoints/bonfires if it is adjusted to reflect the text specific to that waypoint/bonfire."},"comment":{"url":"https://api.github.com/repos/FreeCodeCamp/FreeCodeCamp/issues/comments/168906116","html_url":"https://github.com/FreeCodeCamp/FreeCodeCamp/pull/5880#issuecomment-168906116","issue_url":"https://api.github.com/repos/FreeCodeCamp/FreeCodeCamp/issues/5880","id":168906116,"user":{"login":"SaintPeter","id":553494,"avatar_url":"https://avatars.githubusercontent.com/u/553494?v=3","gravatar_id":"","url":"https://api.github.com/users/SaintPeter","html_url":"https://github.com/SaintPeter","followers_url":"https://api.github.com/users/SaintPeter/followers","following_url":"https://api.github.com/users/SaintPeter/following{/other_user}","gists_url":"https://api.github.com/users/SaintPeter/gists{/gist_id}","starred_url":"https://api.github.com/users/SaintPeter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SaintPeter/subscriptions","organizations_url":"https://api.github.com/users/SaintPeter/orgs","repos_url":"https://api.github.com/users/SaintPeter/repos","events_url":"https://api.github.com/users/SaintPeter/events{/privacy}","received_events_url":"https://api.github.com/users/SaintPeter/received_events","type":"User","site_admin":false},"created_at":"2016-01-05T05:49:44Z","updated_at":"2016-01-05T05:49:44Z","body":"The reason it is failing is because there is an automated test.  The \"solution\" code that I have in the challenge definition does not have comments, so you can't use the regex to find them.\r\n\r\nBTW, did you know that `\\s` also matches `\\n`?  No need for all those `|` bars in there.\r\n\r\nRegardless, I think this is a spectacularly bad idea, in general:  We just don't NEED to check if they're changing the code below the line.  The majority of users do not.  We can check what we can check and devil take the hindmost.\r\n\r\nMy intent was that we clean up the test descriptions so that it was clear that we were talking about the \"final\" values of a,b, and c. "}}