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.

Sign up for GitHub

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

Jump to bottom

Refactor package setup and add tox testing to Azure Pipelines #12

Merged
merged 10 commits into from Jan 16, 2021
Merged

Refactor package setup and add tox testing to Azure Pipelines #12

merged 10 commits into from Jan 16, 2021

Conversation

ConorMacBride
Copy link
Owner

  • Move the details in setup.py into new setup.cfg
  • Fixed some issues with unused dependencies and incompatible min versions
  • Added configuration for tox, to test using pytest and doc building
  • Added multiple conditional testing stages to the Azure Pipelines configuration

@ConorMacBride
Moved setup configuration to setup.cfg
c21190f 
Also changes dependencies: removed docutils (unused); min version 1.3->1.4 scipy (quad_vec not in 1.3.*); min version 0.21->0.22 (py38 support added 0.22).

Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride
Added tox
0dd49ac 
envlist =
  py{36,37,38}{,-oldestdeps}
  build_docs

Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride
Install extra dependencies for tests
7bd93f1 
Signed-off-by: Conor MacBride <conor@macbride.me>
@codecov
Copy link

codecov bot commented Jan 15, 2021 •
edited

Codecov Report

Merging #12 (9488351) into master (d1fdeef) will increase coverage by 0.17%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   70.41%   70.59%   +0.17%     
==========================================
  Files          17       17              
  Lines        1112     1112              
==========================================
+ Hits          783      785       +2     
+ Misses        329      327       -2
Impacted Files Coverage Δ
src/mcalf/models/ibis.py 90.67% <0.00%> (+0.64%) ⬆️

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 d1fdeef...9488351. Read the comment docs.

@ConorMacBride
Configure Azure Pipeline tests
ce13f01 
Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride
Revert to running Azure Pipelines on all branches
b0cbccd 
Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride
Only run on default branch and added missing YAML key
2a9186a 
Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride
Moved build_docs test to StageTwoTests and added pandoc
8f14d7b 
Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride
Moved libraries key to parameters
199aef4 
Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride
Use setuptools_scm to get version from git
1473cc4 
Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride
Get docs version from installed package
9488351 
Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride ConorMacBride merged commit 299e25c into master Jan 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@ConorMacBride