Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

First steps for adding Azure Pipelines #194

Closed
wants to merge 3 commits into from

Conversation

hugovk
Copy link
Contributor

@hugovk hugovk commented Mar 4, 2019

For #193.

Cloned the Travis CI config and updated for Azure Pipelines based on this table of Travis and Azure env vars:

https://docs.microsoft.com/en-us/azure/devops/pipelines/migrate/from-travis?view=azure-devops#predefined-variables

More info on the env vars in each CI:

I've not tested the variables are actually map up as used, they may need some tweaking to make sure you get the right values in the Codecov backend, but I thought I'd make start to help out :)

blueyed added a commit to blueyed/pytest that referenced this pull request Jun 5, 2019
Currently it uses e.g. `branch=HEAD`, which likely confuses our reports
with regard to the base coverage to compare against.

Ref: codecov/codecov-python#194
@blueyed
Copy link
Contributor

blueyed commented Jun 5, 2019

@hugovk
Thanks for this - going to use/try it with pytest: pytest-dev/pytest#5403

@blueyed
Copy link
Contributor

blueyed commented Jun 5, 2019

@blueyed
Copy link
Contributor

blueyed commented Jun 5, 2019

Does not look really good:

.query commit=f49f3484e47e64ba31cbee3e158f08c59cc02c2d&tag=refs%2Fpull%2F5403%2Fmerge&token=&pr=285209934&name=py35-xdist&build=Test%20py35-xdist&package=py2.0.15&branch=refs%2Fpull%2F5403%2Fmerge

  • "tag" is wrong
  • "pr" is wrong
  • "branch" is wrong (the one I want to fix)

I've switched over to codecov-bash for this also: codecov/codecov-bash#148 (comment)

@blueyed
Copy link
Contributor

blueyed commented Jun 5, 2019

According to codecov/codecov-bash#158 (in codecov.io/bash already) service="azure_pipelines" is accepted by the backend already.

@codecov
Copy link

codecov bot commented Jun 5, 2019

Codecov Report

Merging #194 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #194   +/-   ##
=====================================
  Coverage       0%     0%           
=====================================
  Files           1      1           
  Lines           1      1           
=====================================
  Misses          1      1

@hugovk
Copy link
Contributor Author

hugovk commented Jun 5, 2019

Thanks, updated!

ofek added a commit to ofek/codecov-python that referenced this pull request Sep 12, 2019
@hugovk hugovk closed this May 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants