Skip to content

Conversation

@chfw
Copy link
Member

@chfw chfw commented Mar 6, 2019

strengthens #180

single cli output has been address previously. this PR join legacy copy targets with normal and only targets so that they are executed in the same pipe line.

what's more, code refactoring is done so that cli target is seen as top priority.

chfw added 2 commits March 6, 2019 07:45
…s. v0.4.1 or earlier, copy targets are executed in a separate handler, meaning at least the counter, the number of templated files are reset, and there will be two stastistics. And this change make cli target a clear priority. fix #180
@chfw chfw requested review from CLiu13 and ayan-b March 6, 2019 07:51
@ayan-b
Copy link
Member

ayan-b commented Mar 6, 2019

CI fails, restarting the job.
UPD: Now it passes.

@codecov-io
Copy link

codecov-io commented Mar 6, 2019

Codecov Report

Merging #248 into dev will decrease coverage by 0.04%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             dev     #248      +/-   ##
=========================================
- Coverage   99.3%   99.26%   -0.05%     
=========================================
  Files         57       57              
  Lines       2435     2434       -1     
=========================================
- Hits        2418     2416       -2     
- Misses        17       18       +1
Impacted Files Coverage Δ
moban/mobanfile/__init__.py 99.23% <100%> (-0.01%) ⬇️
moban/plugins/context.py 94.73% <0%> (-5.27%) ⬇️

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 5133306...d82016c. Read the comment docs.

@chfw chfw closed this Mar 6, 2019
@chfw chfw deleted the single-cli-output branch March 6, 2019 18:46
@chfw chfw restored the single-cli-output branch March 6, 2019 18:48
@chfw chfw reopened this Mar 6, 2019
@chfw chfw merged commit 6a2582b into dev Mar 6, 2019
@chfw chfw deleted the single-cli-output branch March 6, 2019 18:48
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