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.

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

feat(cli): init with webapp dist #35

Merged
merged 2 commits into from
Dec 21, 2021
Merged

Conversation

Eliot00
Copy link
Contributor

@Eliot00 Eliot00 commented Dec 21, 2021

下载web.zip解压后得到的文件目录是dist,web_server默认的dist目录是web,这两个名称要统一下

@Eliot00
Copy link
Contributor Author

Eliot00 commented Dec 21, 2021

fix issues #19

@codecov-commenter
Copy link

Codecov Report

Merging #35 (1c4b069) into master (b1ca356) will decrease coverage by 0.89%.
The diff coverage is 17.74%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
- Coverage   74.43%   73.53%   -0.90%     
==========================================
  Files          62       62              
  Lines        4439     4496      +57     
==========================================
+ Hits         3304     3306       +2     
- Misses       1135     1190      +55     
Impacted Files Coverage Δ
src/main.rs 46.00% <17.74%> (-12.55%) ⬇️

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 b1ca356...1c4b069. Read the comment docs.

@phodal
Copy link
Owner

phodal commented Dec 21, 2021

@Eliot00 后续改一下自动化构建那个吧,改为 web

@Eliot00 Eliot00 marked this pull request as ready for review December 21, 2021 09:57
@phodal phodal merged commit 01c2e72 into phodal:master Dec 21, 2021
@Eliot00 Eliot00 deleted the init-project branch December 24, 2021 07:18
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.

None yet

3 participants