Q
[axios]修复#211
8
A
回答
1
@dublx 你能更新一下 README.md 吗?
6
你好,我对 PR 不太有经验..我是否更新自述文件,然后提交新的?不管怎样,我改变最大代码的原因是因为这样承诺就会被拒绝。然后我会改变它:检查 if maxRedirects === 0
,想法是不应该有重定向,但也不应该有承诺拒绝。同意?
0
您可以将新的提交推送到同一分支。它将自动添加到您的 PR 中。
至于最大代码,我的建议是根本不要更改状态代码错误范围。如果您不希望 axios 拒绝 3XX 响应的承诺,则可以validateStatus
在合并 #308 时通过配置选项来做到这一点。
5
@dublx 如果您需要任何帮助,请告诉我
3
@nickuraltsev 你好,我非常喜欢在我为公司创建的聊天机器人中使用 axios。我需要这个 PR 来支持一些所需的功能,目前计划分叉我自己的副本,直到它可用。如果 @dublx 没问题,我会很乐意清理这个分支,以不更改状态代码错误范围,更新自述文件并修复任何冲突。或者我可以简单地创建一个新的 PR。
简而言之,我希望看到我即将要做的工作被放回到 axios 中与其他人分享,这样我就不必维护自己的 fork。如果我可以提供帮助,请告诉我。
8
@cspray 是的,我认为处理这个问题的正确方法是将 @dublx 提交合并到您自己的分支中,进行必要的更改,然后提交新的 PR。如果你能做到这一点,那就太好了。谢谢你!
2
@nickuraltsev 会的!
0
大家好@dublx @nickuraltsev @cspray @mzabriskie,这个 PR 有可能很快被合并吗?我真的很期待:)我可以帮忙吗?干杯:)
4
合并