timfeirg commented 2 years ago

closes https://github.com/timfeirg/lain-cli/issues/64

timfeirg commented 2 years ago

CI 失败是由于 python 3.8 尚不支持 dict merge operator (|), 因此正在调研将 lain 的最低 python 要求设置为 3.9. 其实就是问问大伙迁移有无障碍.

hongqn commented 2 years ago

我这里 python 3.9 没问题。

codecov-commenter commented 2 years ago

Codecov Report

Merging #65 (5bafba1) into master (ce88285) will increase coverage by 0.38%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
+ Coverage   55.30%   55.68%   +0.38%     
==========================================
  Files          14       14              
  Lines        4045     4069      +24     
==========================================
+ Hits         2237     2266      +29     
+ Misses       1808     1803       -5

Impacted Files	Coverage Δ
lain_cli/utils.py	`71.04% <100.00%> (+0.72%)`	:arrow_up:
lain_cli/prompt.py	`16.90% <0.00%> (-0.13%)`	:arrow_down:
lain_cli/lain.py	`51.77% <0.00%> (-0.05%)`	:arrow_down:

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 ce88285...5bafba1. Read the comment docs.

timfeirg commented 2 years ago

实现比较糙, 按理说应该分阶段 (build stage) 来控制 args, 但我想目前也没啥需求, 就先做成这样吧...

timfeirg / lain-cli

[lain build] directly use system env #65

Codecov Report