Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reorganize azure testing #1080

Merged
merged 90 commits into from
Feb 21, 2020
Merged

Reorganize azure testing #1080

merged 90 commits into from
Feb 21, 2020

Conversation

ahartikainen
Copy link
Contributor

@ahartikainen ahartikainen commented Feb 19, 2020

Description

Implements all the tests and builds on Azure. Removes Travis. Fixes #1061 and fixes #1060

Python 3.6, 3.7, 3.8

  • BaseTests --> Tests ArviZ based tests (stats, diagnostics, plots)
  • ExternalTests --> Test external libraries (data)
  • Run benchmarks --> Upload results to pipeline site

Python 3.8

  • Build Docs (build only if tests are passed and on master branch)
  • Build Wheel (build only if tests are passed and a tag "branch") --> upload dist folder to pipeline site --> can be download and upload with twine for pypi manually.

Changes coveralls to codecov.

Checklist

Future

We could follow benchmarks somehow, but currently they are only there so we can follow them manually.

requirements-docs.txt Outdated Show resolved Hide resolved
mytests.txt Outdated Show resolved Hide resolved
@OriolAbril
Copy link
Member

This is probably a good time to remove the azurepipelines-coverage file which doesn't actually do anything.

@ahartikainen
Copy link
Contributor Author

Cool,

Still needs the docs

.azure-pipelines/azure-pipelines-docs.yml

Some fix for coverage and pytest reports.

Also reporting lint etc?

Codecov? We probably should also skip tests folder from coverage.

I dropped some tests (saving external file)

@codecov-io
Copy link

codecov-io commented Feb 20, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@3c2bd28). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1080   +/-   ##
=========================================
  Coverage          ?   91.61%           
=========================================
  Files             ?       93           
  Lines             ?     8704           
  Branches          ?        0           
=========================================
  Hits              ?     7974           
  Misses            ?      730           
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c2bd28...3a6aae8. Read the comment docs.

@ahartikainen ahartikainen changed the title [WIP] reorganize azure testing Reorganize azure testing Feb 21, 2020
@OriolAbril OriolAbril merged commit d544c09 into master Feb 21, 2020
@ahartikainen ahartikainen deleted the update_azure branch February 25, 2020 14:43
@canyon289
Copy link
Member

I am very late to this PR but just wanted to say the work here looks great! Was looking at the testing now and everything is much better

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add coverage check in PRs Build docs in Azure
4 participants