Skip to content

添加isChangingConfigurations() - #32

Merged
shifujun merged 1 commit into
Tencent:devfrom
SheepYang1993:add_isChangingConfigurations
Jul 12, 2019
Merged

添加isChangingConfigurations()#32
shifujun merged 1 commit into
Tencent:devfrom
SheepYang1993:add_isChangingConfigurations

Conversation

@SheepYang1993

Copy link
Copy Markdown
Contributor

这个提交吗,大牛review一下:joy:

@tencent-adm

tencent-adm commented Jul 4, 2019

Copy link
Copy Markdown
Member

CLA assistant check
All committers have signed the CLA.

@shifujun

shifujun commented Jul 4, 2019

Copy link
Copy Markdown
Collaborator

可以呀。这种没有实现的方法欢迎大家补充。

不过你能不能接受稍微提高些门槛?因为我们没有直接实现全所有方法,也是因为那些方法我们平时没用到,所以也不熟悉那些API。所以下面这几点,最好能尝试做到。

  1. 能不能简单说明一下这个API的功能,解释一下为什么这样简单转调就是科学的。
  2. 在Sample中添加一个使用这个API的例子。让我们能看到正常安装运行和插件环境运行下行为是一致的。
  3. 在Test中添加一个自动化测试用例。

我不会要求你都做到,看看不能尝试做到呢?如果嫌麻烦的话,也可以合进来现在的代码,参与提交代码本身是最重要的。

@SheepYang1993

SheepYang1993 commented Jul 4, 2019

Copy link
Copy Markdown
Contributor Author

我不会要求你都做到,看看不能尝试做到呢?如果嫌麻烦的话,也可以合进来现在的代码,参与提交代码本身是最重要的。

好的,我有空尝试添加一下,只是不知道sample和test需不需要啥格式 或者 标准。
功能虽然简单,关键是要有开源的氛围,哈哈~:kissing_heart:

ps:代码注释可以PR吗?作为小白要入手框架,没有注释还是有点难度的。。。

@shifujun

shifujun commented Jul 5, 2019

Copy link
Copy Markdown
Collaborator

好像你编辑回复,我不会收到通知。下回直接再次回复吧。

代码标准未来会考虑,目前重要的是有人参与,所以门槛会放低的。

sample也没什么标准,基本上照抄现有的就好了。

sample实际上是从test复制出来的,因为我们发现test中有很多用例是测试所必须的,但是对于功能演示来说重复度太高太无聊了。test的发展方向将会是去掉人工操作界面,去掉xml等资源,以简洁的代码实现,完全只支持用AndroidTest自动化运行。

代码注释当然欢迎PR了,只要注释的内容没错误,不会太要求格式的。但整洁规范总是更好的。还是不希望把参与的门槛搞太高。

@shifujun

shifujun commented Jul 5, 2019

Copy link
Copy Markdown
Collaborator

另外,如果有其他人看到。我想说,不是Shadow不欢迎小白,而是不管维护Shadow还是使用Shadow都有天然客观的技术门槛

要降低这种门槛需要大量的设计和更详尽的文档,甚至是一些技术的普及教程。所以,希望自认小白的同学有些耐心,只要是具体问题,都会尽量解答的。但是像“怎么一键接入”这种太难短时间内实现的问题,就不好解决了。

@shifujun
shifujun merged commit d100330 into Tencent:dev Jul 12, 2019
@shifujun

Copy link
Copy Markdown
Collaborator

先合进来了。

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.

3 participants