{"action":"created","issue":{"url":"https://api.github.com/repos/codecov/codecov-python/issues/68","repository_url":"https://api.github.com/repos/codecov/codecov-python","labels_url":"https://api.github.com/repos/codecov/codecov-python/issues/68/labels{/name}","comments_url":"https://api.github.com/repos/codecov/codecov-python/issues/68/comments","events_url":"https://api.github.com/repos/codecov/codecov-python/issues/68/events","html_url":"https://github.com/codecov/codecov-python/issues/68","id":162969093,"number":68,"title":"Agnostic support for CI systems","user":{"login":"adiroiban","id":204609,"avatar_url":"https://avatars.githubusercontent.com/u/204609?v=3","gravatar_id":"","url":"https://api.github.com/users/adiroiban","html_url":"https://github.com/adiroiban","followers_url":"https://api.github.com/users/adiroiban/followers","following_url":"https://api.github.com/users/adiroiban/following{/other_user}","gists_url":"https://api.github.com/users/adiroiban/gists{/gist_id}","starred_url":"https://api.github.com/users/adiroiban/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adiroiban/subscriptions","organizations_url":"https://api.github.com/users/adiroiban/orgs","repos_url":"https://api.github.com/users/adiroiban/repos","events_url":"https://api.github.com/users/adiroiban/events{/privacy}","received_events_url":"https://api.github.com/users/adiroiban/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-06-29T16:24:11Z","updated_at":"2016-07-10T16:03:09Z","closed_at":null,"body":"This is a follow up for https://github.com/buildbot/buildbot/pull/1671\r\n\r\nI see the code from https://github.com/codecov/codecov-python/blob/master/codecov/__init__.py#L260-L430 is trying to hunt each CI system and extract the required information .\r\n\r\nI don't see any generic lookup for those environment variables ... other than the token.\r\n\r\nThose variables should also be VCS agnostic ... but for now this is of lower priority, since only GitHub/GitLab and Bitbucket are supported anyway.\r\n\r\nThe goal is to allow sending reports from Buildbot or other CI system without requiring to update codecov-python for each new CI system.\r\n\r\nFor my project I am using this environment in Buildbot https://github.com/chevah/buildbot-configuration-builder/blob/master/chevah/buildbot_configuration_builder/builder.py#L859\r\n\r\n-----------\r\n\r\nAs agnostic environment variables I suggest:\r\n\r\ncommit -> VCS_COMMIT_ID\r\nbranch -> VCS_BRANCH_NAME\r\nbranch_url -> VCS_BRANCH_URL\r\nbuild -> VCS_BUILD_ID\r\nbuild_url -> VCS_BUILD_URL\r\npr -> GITHUB_PR_ID | GITLAB_PR_ID\r\nslug -> GITHUB_SLUG | GITLAB_SLUG\r\n\r\n----------\r\n\r\nany CI system should set `CI=true` and maybe 'CI_NAME=service`"},"comment":{"url":"https://api.github.com/repos/codecov/codecov-python/issues/comments/231596401","html_url":"https://github.com/codecov/codecov-python/issues/68#issuecomment-231596401","issue_url":"https://api.github.com/repos/codecov/codecov-python/issues/68","id":231596401,"user":{"login":"adiroiban","id":204609,"avatar_url":"https://avatars.githubusercontent.com/u/204609?v=3","gravatar_id":"","url":"https://api.github.com/users/adiroiban","html_url":"https://github.com/adiroiban","followers_url":"https://api.github.com/users/adiroiban/followers","following_url":"https://api.github.com/users/adiroiban/following{/other_user}","gists_url":"https://api.github.com/users/adiroiban/gists{/gist_id}","starred_url":"https://api.github.com/users/adiroiban/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adiroiban/subscriptions","organizations_url":"https://api.github.com/users/adiroiban/orgs","repos_url":"https://api.github.com/users/adiroiban/repos","events_url":"https://api.github.com/users/adiroiban/events{/privacy}","received_events_url":"https://api.github.com/users/adiroiban/received_events","type":"User","site_admin":false},"created_at":"2016-07-10T16:03:09Z","updated_at":"2016-07-10T16:03:09Z","body":"I think that instead of\r\n\r\n```\r\nbuild -> VCS_BUILD_ID\r\nbuild_url -> VCS_BUILD_URL\r\n```\r\n\r\nwe should have\r\n\r\n```\r\nbuild -> CI_BUILD_ID\r\nbuild_url -> CI_BUILD_URL\r\n```\r\n\r\nas these variables are related to the CI system and not  with the VCS"}}
{"actor":{"display_login":"adiroiban"}}