{"action":"created","issue":{"url":"https://api.github.com/repos/jenkinsci/scm-trait-commit-skip-plugin/issues/13","repository_url":"https://api.github.com/repos/jenkinsci/scm-trait-commit-skip-plugin","labels_url":"https://api.github.com/repos/jenkinsci/scm-trait-commit-skip-plugin/issues/13/labels{/name}","comments_url":"https://api.github.com/repos/jenkinsci/scm-trait-commit-skip-plugin/issues/13/comments","events_url":"https://api.github.com/repos/jenkinsci/scm-trait-commit-skip-plugin/issues/13/events","html_url":"https://github.com/jenkinsci/scm-trait-commit-skip-plugin/pull/13","id":377641163,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI4NTM2ODA0","number":13,"title":"Add build strategy to skip events that trigger a build if HEAD commit matches some rules (continued)","user":{"login":"mryan43","id":223869,"node_id":"MDQ6VXNlcjIyMzg2OQ==","avatar_url":"https://avatars0.githubusercontent.com/u/223869?v=4","gravatar_id":"","url":"https://api.github.com/users/mryan43","html_url":"https://github.com/mryan43","followers_url":"https://api.github.com/users/mryan43/followers","following_url":"https://api.github.com/users/mryan43/following{/other_user}","gists_url":"https://api.github.com/users/mryan43/gists{/gist_id}","starred_url":"https://api.github.com/users/mryan43/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mryan43/subscriptions","organizations_url":"https://api.github.com/users/mryan43/orgs","repos_url":"https://api.github.com/users/mryan43/repos","events_url":"https://api.github.com/users/mryan43/events{/privacy}","received_events_url":"https://api.github.com/users/mryan43/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-11-06T00:26:43Z","updated_at":"2018-11-09T13:19:21Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/jenkinsci/scm-trait-commit-skip-plugin/pulls/13","html_url":"https://github.com/jenkinsci/scm-trait-commit-skip-plugin/pull/13","diff_url":"https://github.com/jenkinsci/scm-trait-commit-skip-plugin/pull/13.diff","patch_url":"https://github.com/jenkinsci/scm-trait-commit-skip-plugin/pull/13.patch"},"body":"This is a follow up on PR #10  by @nfalco79 with changes requested by @witokondoria (kudos to both of them for maintaining/contributing to this plugin !) I have added:\r\n- Github implementation\r\n- Separation of commit author and message filtering into different BranchBuildStrategy classes\r\n- Usage of regex instead of wildcards\r\n- Common logic extracted into parent maven module\r\n\r\nI have tested it on both github.com and bitbucket cloud (it should in theory also work with github enterprise and bitbucket server)\r\n\r\nKnow issue : https://issues.jenkins-ci.org/browse/JENKINS-54468\r\nThis only affects users who have both the github and bitbucket flavors of the plugin and there is a simple workaround for affected users (only use the applicable branch build strategy in the UI). I don't think this should be a blocker to merge this PR as the fix would have to be done in the branch-api plugin anyway.\r\n\r\n<!-- Reviewable:start -->\r\n---\r\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/jenkinsci/scm-trait-commit-skip-plugin/13)\r\n<!-- Reviewable:end -->\r\n"},"comment":{"url":"https://api.github.com/repos/jenkinsci/scm-trait-commit-skip-plugin/issues/comments/437357061","html_url":"https://github.com/jenkinsci/scm-trait-commit-skip-plugin/pull/13#issuecomment-437357061","issue_url":"https://api.github.com/repos/jenkinsci/scm-trait-commit-skip-plugin/issues/13","id":437357061,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNzM1NzA2MQ==","user":{"login":"mryan43","id":223869,"node_id":"MDQ6VXNlcjIyMzg2OQ==","avatar_url":"https://avatars0.githubusercontent.com/u/223869?v=4","gravatar_id":"","url":"https://api.github.com/users/mryan43","html_url":"https://github.com/mryan43","followers_url":"https://api.github.com/users/mryan43/followers","following_url":"https://api.github.com/users/mryan43/following{/other_user}","gists_url":"https://api.github.com/users/mryan43/gists{/gist_id}","starred_url":"https://api.github.com/users/mryan43/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mryan43/subscriptions","organizations_url":"https://api.github.com/users/mryan43/orgs","repos_url":"https://api.github.com/users/mryan43/repos","events_url":"https://api.github.com/users/mryan43/events{/privacy}","received_events_url":"https://api.github.com/users/mryan43/received_events","type":"User","site_admin":false},"created_at":"2018-11-09T13:19:21Z","updated_at":"2018-11-09T13:19:21Z","author_association":"NONE","body":"I found a bug in the github implementation, please wait until I fix it before merging this PR"}}
{"actor":{"display_login":"mryan43"}}