Skip to content

Fix issues in the UTs and workflow#229

Merged
amcadmus merged 10 commits into
deepmodeling:develfrom
amcadmus:fix-uts-workflow
Dec 1, 2021
Merged

Fix issues in the UTs and workflow#229
amcadmus merged 10 commits into
deepmodeling:develfrom
amcadmus:fix-uts-workflow

Conversation

@wanghan-iapcm

Copy link
Copy Markdown
Contributor
  • skip the UTs on machines that do not have rdkit
  • only mirror to gitee if the org is deepmodeling.

@wanghan-iapcm wanghan-iapcm requested a review from njzjz November 25, 2021 02:55
@codecov-commenter

codecov-commenter commented Nov 25, 2021

Copy link
Copy Markdown

Codecov Report

Merging #229 (44c616a) into devel (a2d13e6) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            devel     #229   +/-   ##
=======================================
  Coverage   82.04%   82.04%           
=======================================
  Files          58       58           
  Lines        5034     5034           
=======================================
  Hits         4130     4130           
  Misses        904      904           

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 a2d13e6...44c616a. Read the comment docs.

Comment thread tests/test_bond_order_system.py
Comment thread tests/test_amber_sqm.py
amcadmus and others added 2 commits December 1, 2021 08:03
Co-authored-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Co-authored-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@amcadmus amcadmus merged commit f639f95 into deepmodeling:devel Dec 1, 2021
@amcadmus amcadmus deleted the fix-uts-workflow branch February 26, 2022 12:26
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.

4 participants