Skip to content

refactor!: cmd.core -> utils.run#343

Merged
tony merged 2 commits into
masterfrom
move-run
May 15, 2022
Merged

refactor!: cmd.core -> utils.run#343
tony merged 2 commits into
masterfrom
move-run

Conversation

@tony

@tony tony commented May 15, 2022

Copy link
Copy Markdown
Member

Move to cmd.core -> utils.run. This is not to be considered a stable API in the near future - though we'd like to after it'd battle-tested.

@codecov

codecov Bot commented May 15, 2022

Copy link
Copy Markdown

Codecov Report

Merging #343 (29012ee) into master (01643ce) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 29012ee differs from pull request most recent head 85dba41. Consider uploading reports for the commit 85dba41 to get more accurate results

@@           Coverage Diff           @@
##           master     #343   +/-   ##
=======================================
  Coverage   74.80%   74.80%           
=======================================
  Files          26       26           
  Lines        1961     1961           
=======================================
  Hits         1467     1467           
  Misses        494      494           
Impacted Files Coverage Δ
libvcs/utils/run.py 89.61% <ø> (ø)
libvcs/cmd/git.py 53.34% <100.00%> (ø)
libvcs/cmd/hg.py 69.73% <100.00%> (ø)
libvcs/cmd/svn.py 52.23% <100.00%> (ø)
libvcs/projects/base.py 100.00% <100.00%> (ø)
tests/cmd/test_core.py 100.00% <100.00%> (ø)
tests/projects/test_conftest.py 100.00% <100.00%> (ø)
tests/projects/test_git.py 99.38% <100.00%> (ø)
tests/projects/test_hg.py 95.23% <100.00%> (ø)
tests/projects/test_svn.py 96.15% <100.00%> (ø)

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 01643ce...85dba41. Read the comment docs.

@tony tony merged commit 7a99e5e into master May 15, 2022
@tony tony deleted the move-run branch May 15, 2022 14:55
@tony tony added this to the v0.13 milestone May 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant