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

fix: generate file only when file content changes #3410

Merged
merged 1 commit into from Oct 12, 2019
Merged

Conversation

ikobe-zz
Copy link

Checklist
  • npm test passes
  • tests are included
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

文件内容改变时才重新生成临时文件

@netlify
Copy link

netlify bot commented Oct 11, 2019

Ant Design Pro preview using the current branch umi

Built with commit 51409cc

https://deploy-preview-3410--pro-pre-umijs.netlify.com

@ikobe-zz ikobe-zz changed the title fix: generate file only when file content changes [WIP] fix: generate file only when file content changes Oct 11, 2019
@ikobe-zz
Copy link
Author

routes.js 的内容,即使路由不变,内容也会不一样。需要排查一下

@coveralls
Copy link

coveralls commented Oct 12, 2019

Pull Request Test Coverage Report for Build 7907

  • 0 of 6 (0.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.03%) to 38.894%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/umi-build-dev/src/Service.js 0 1 0.0%
packages/umi-build-dev/src/FilesGenerator.js 0 2 0.0%
packages/umi-build-dev/src/utils/writeContent.js 0 3 0.0%
Totals Coverage Status
Change from base Build 7901: -0.03%
Covered Lines: 4870
Relevant Lines: 12396

💛 - Coveralls

@ikobe-zz ikobe-zz changed the title [WIP] fix: generate file only when file content changes fix: generate file only when file content changes Oct 12, 2019
@sorrycc sorrycc merged commit 85b3f86 into master Oct 12, 2019
@delete-merged-branch delete-merged-branch bot deleted the fixFileGenerator branch October 12, 2019 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants