{"action":"closed","issue":{"url":"https://api.github.com/repos/jneen/parsimmon/issues/24","repository_url":"https://api.github.com/repos/jneen/parsimmon","labels_url":"https://api.github.com/repos/jneen/parsimmon/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/jneen/parsimmon/issues/24/comments","events_url":"https://api.github.com/repos/jneen/parsimmon/issues/24/events","html_url":"https://github.com/jneen/parsimmon/issues/24","id":31293391,"number":24,"title":"Add match group parameter to regexp()","user":{"login":"Bartvds","id":685984,"avatar_url":"https://avatars.githubusercontent.com/u/685984?v=3","gravatar_id":"","url":"https://api.github.com/users/Bartvds","html_url":"https://github.com/Bartvds","followers_url":"https://api.github.com/users/Bartvds/followers","following_url":"https://api.github.com/users/Bartvds/following{/other_user}","gists_url":"https://api.github.com/users/Bartvds/gists{/gist_id}","starred_url":"https://api.github.com/users/Bartvds/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Bartvds/subscriptions","organizations_url":"https://api.github.com/users/Bartvds/orgs","repos_url":"https://api.github.com/users/Bartvds/repos","events_url":"https://api.github.com/users/Bartvds/events{/privacy}","received_events_url":"https://api.github.com/users/Bartvds/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2014-04-11T00:13:02Z","updated_at":"2016-05-20T02:12:20Z","closed_at":"2016-05-20T02:12:20Z","body":"Currently the regexp parser returns the whole match, while it would be very convenient if we could specify a match group.\r\n\r\nGood for optional stuff in the RegExp: so if you have like `/v?(\\d+)/` you could get just the digits and ignore the 'v' prefix.\r\n\r\nI have this working [here](https://github.com/Bartvds/parsimmon/blob/46b08b9da0fa6b03af663021703ca8d7c70d2275/src/parsimmon.js#L266-L298). (looks funny because it optimises a conditional). Got some tests too.\r\n\r\nI can PR this but since you guys are working on parsimmon I'd send a ticket first as I worked on the old error-throwing code."}}
{"actor":{"display_login":"wavebeem"}}