{"action":"created","issue":{"url":"https://api.github.com/repos/conda-forge/staged-recipes/issues/1419","repository_url":"https://api.github.com/repos/conda-forge/staged-recipes","labels_url":"https://api.github.com/repos/conda-forge/staged-recipes/issues/1419/labels{/name}","comments_url":"https://api.github.com/repos/conda-forge/staged-recipes/issues/1419/comments","events_url":"https://api.github.com/repos/conda-forge/staged-recipes/issues/1419/events","html_url":"https://github.com/conda-forge/staged-recipes/pull/1419","id":173932656,"number":1419,"title":"Skip AppVeyor on merged pull requests","user":{"login":"jakirkham","id":3019665,"avatar_url":"https://avatars.githubusercontent.com/u/3019665?v=3","gravatar_id":"","url":"https://api.github.com/users/jakirkham","html_url":"https://github.com/jakirkham","followers_url":"https://api.github.com/users/jakirkham/followers","following_url":"https://api.github.com/users/jakirkham/following{/other_user}","gists_url":"https://api.github.com/users/jakirkham/gists{/gist_id}","starred_url":"https://api.github.com/users/jakirkham/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jakirkham/subscriptions","organizations_url":"https://api.github.com/users/jakirkham/orgs","repos_url":"https://api.github.com/users/jakirkham/repos","events_url":"https://api.github.com/users/jakirkham/events{/privacy}","received_events_url":"https://api.github.com/users/jakirkham/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/conda-forge/staged-recipes/labels/maintenance","name":"maintenance","color":"fbca04"}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-08-30T05:26:25Z","updated_at":"2016-08-30T05:34:11Z","closed_at":"2016-08-30T05:31:23Z","pull_request":{"url":"https://api.github.com/repos/conda-forge/staged-recipes/pulls/1419","html_url":"https://github.com/conda-forge/staged-recipes/pull/1419","diff_url":"https://github.com/conda-forge/staged-recipes/pull/1419.diff","patch_url":"https://github.com/conda-forge/staged-recipes/pull/1419.patch"},"body":"The goal here is to only run Travis CI when merging a pull request. As Travis CI does the feedstock conversion on `master` after a merge, it is the only CI that is relevant for running. The other CIs exist only to build PRs. We already successfully have disabled CircleCI on `master` with PR ( https://github.com/conda-forge/staged-recipes/pull/329 ). However, it has proven tricky to do the same with AppVeyor. A very similar PR ( https://github.com/conda-forge/staged-recipes/pull/1418 ) for AppVeyor that applied the strategy from CircleCI failed completely at the PR level. So had to be abandoned.\r\n\r\nThis tries to skip AppVeyor builds if they are a merged pull request. The way it works is it searches for the message that a merged pull request would have (i.e. `Merge pull request `). If it finds such a message at the beginning of the commit, it simply does not run CI with AppVeyor on that commit.\r\n\r\nIn order to test this, it will need to be merged. However, if it doesn't work or has issues, would totally be ok ripping it out. Given that it is really quite simple and potentially useful, am hopeful there is no need for that. "},"comment":{"url":"https://api.github.com/repos/conda-forge/staged-recipes/issues/comments/243337485","html_url":"https://github.com/conda-forge/staged-recipes/pull/1419#issuecomment-243337485","issue_url":"https://api.github.com/repos/conda-forge/staged-recipes/issues/1419","id":243337485,"user":{"login":"jakirkham","id":3019665,"avatar_url":"https://avatars.githubusercontent.com/u/3019665?v=3","gravatar_id":"","url":"https://api.github.com/users/jakirkham","html_url":"https://github.com/jakirkham","followers_url":"https://api.github.com/users/jakirkham/followers","following_url":"https://api.github.com/users/jakirkham/following{/other_user}","gists_url":"https://api.github.com/users/jakirkham/gists{/gist_id}","starred_url":"https://api.github.com/users/jakirkham/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jakirkham/subscriptions","organizations_url":"https://api.github.com/users/jakirkham/orgs","repos_url":"https://api.github.com/users/jakirkham/repos","events_url":"https://api.github.com/users/jakirkham/events{/privacy}","received_events_url":"https://api.github.com/users/jakirkham/received_events","type":"User","site_admin":false},"created_at":"2016-08-30T05:34:11Z","updated_at":"2016-08-30T05:34:11Z","body":"Went ahead and merged as the CIs weren't relevant for testing this per se. Though seeing that they all started (including AppVeyor) was worthwhile. It does appear that no run on AppVeyor was triggered on `master` after merging this. So this does appear to work."}}
{"actor":{"display_login":"jakirkham"}}