Closed AntonioShare closed 1 year ago
感谢贡献代码。这个PR看起来很干净,应该很好合入。目前CI检查已经有点报错。你可以自己修改提交,或者我可以晚些帮你改好合入。
CI会检查每个提交,而不仅仅是最终状态。所以你需要rebase提交,然后force push。
CI会检查每个提交,而不仅仅是最终状态。所以你需要rebase提交,然后force push。
现在应该可以了
CI会检查每个提交,而不仅仅是最终状态。所以你需要rebase提交,然后force push。
现在应该可以了
你看这个PR还是包含两个提交,这应该是不对的。你应该把第二个fix提交合并到最初的提交中。同时修改最初的提交的commit message,使其通过检查。code format检查确实是只检查最终代码,但是commit message检查是要检查每一个提交的。
CI会检查每个提交,而不仅仅是最终状态。所以你需要rebase提交,然后force push。
非常抱歉,似乎还有问题,我将再次检查一下代码在进行force push
CI会检查每个提交,而不仅仅是最终状态。所以你需要rebase提交,然后force push。
现在应该可以了
你看这个PR还是包含两个提交,这应该是不对的。你应该把第二个fix提交合并到最初的提交中。同时修改最初的提交的commit message,使其通过检查。code format检查确实是只检查最终代码,但是commit message检查是要检查每一个提交的。
好的,非常抱歉。我这就更改
另外你如果愿意自己动手的话,也可以在自己fork的仓库开启github action。这样可以在自己的仓库触发同样的CI检查。
另外你如果愿意自己动手的话,也可以在自己fork的仓库开启github action。这样可以在自己的仓库触发同样的CI检查。
好的,最后一次我已经检验过了,再次感谢您的指点
另外你如果愿意自己动手的话,也可以在自己fork的仓库开启github action。这样可以在自己的仓库触发同样的CI检查。
非常抱歉,最后一次多了一个.字符忘记删,已修正
另外你如果愿意自己承担的话,也可以在自己fork的仓库开启github操作。这样就可以在自己的仓库触发同样的CI检查。
检测已经通过,代码似乎还需您review一下才能进去
done🎉
Support setting the horizontal and vertical screen orientation in the plugin's AndroidManifest. And set the TestActivityOrientation direction in AndroidManifest to landscape