This repository was archived by the owner on Dec 20, 2017. It is now read-only.
允许通过 .leanengineignore 定义部署时不上传的文件#121
Merged
aisk merged 1 commit intoleancloud:masterfrom Jan 23, 2016
jysperm:ignore-files
Merged
允许通过 .leanengineignore 定义部署时不上传的文件#121aisk merged 1 commit intoleancloud:masterfrom jysperm:ignore-files
aisk merged 1 commit intoleancloud:masterfrom
jysperm:ignore-files
Conversation
Contributor
|
不是很确定 .npmignore 要不要支持,因为我们本来就支持多个运行时环境,只支持 node 有点奇怪。 |
Contributor
Author
|
忘记这事了,看来这个也要做成可配置的 |
Contributor
|
我觉得得考虑不同语言的 ignore,我们自己的 ignore,git 的 ignore,太复杂了,不如在 deploy 的时候接受一个参数 |
Contributor
Author
|
@aisk 那就先只支持 |
Contributor
|
LGTM |
aisk
added a commit
that referenced
this pull request
Jan 23, 2016
允许通过 .leanengineignore 定义部署时不上传的文件
Contributor
Author
|
补充一下典型用法,例如新建一个
|
Contributor
|
我们最终是希望通过 rsync 来进行部署,所以需要确认下 rsync 支持的 exclude pattern |
Contributor
Author
|
大概看了一下,语法很接近,但细节上可能还是有不兼容的地方的,我觉得可以先发布这个功能,在文档上只介绍最基本的写法,或者可以暂时不写在文档上,因为只是很少一部分规模大或者构建流程复杂的用户才需要。 |
Contributor
Author
|
https://github.com/isaacs/node-glob 从描述看,rsync 更复杂,我们支持的是 rsync 的一个子集。 |
Contributor
|
每次部署都要3~4分钟且不一定能成功的我只好在这里默默的点个赞 话说忽略任意层级下的node_modules 会常用吗? |
Contributor
|
@GongT 这个问题是之前实现导致的,目前在升级新版,新版可以提高部署速度。这个 issue 是 ignore 的功能。 |
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#109 @aisk @wangxiao