[axios]修复#211

2024-05-15 39 views

回答

5

@dublx 你能更新一下 README.md 吗?

9

你好,我对 PR 不太有经验..我是否更新自述文件,然后提交新的?不管怎样,我改变最大代码的原因是因为这样承诺就会被拒绝。然后我会改变它:检查 if maxRedirects === 0,想法是不应该有重定向,但也不应该有承诺拒绝。同意?

8

您可以将新的提交推送到同一分支。它将自动添加到您的 PR 中。

至于最大代码,我的建议是根本不要更改状态代码错误范围。如果您不希望 axios 拒绝 3XX 响应的承诺,则可以validateStatus在合并 #308 时通过配置选项来做到这一点。

9

@dublx 如果您需要任何帮助,请告诉我

6

@nickuraltsev 你好,我非常喜欢在我为公司创建的聊天机器人中使用 axios。我需要这个 PR 来支持一些所需的功能,目前计划分叉我自己的副本,直到它可用。如果 @dublx 没问题,我会很乐意清理这个分支,以不更改状态代码错误范围,更新自述文件并修复任何冲突。或者我可以简单地创建一个新的 PR。

简而言之,我希望看到我即将要做的工作被放回到 axios 中与其他人分享,这样我就不必维护自己的 fork。如果我可以提供帮助,请告诉我。

2

@cspray 是的,我认为处理这个问题的正确方法是将 @dublx 提交合并到您自己的分支中,进行必要的更改,然后提交新的 PR。如果你能做到这一点,那就太好了。谢谢你!

0

@nickuraltsev 会的!

3

大家好@dublx @nickuraltsev @cspray @mzabriskie,这个 PR 有可能很快被合并吗?我真的很期待:)我可以帮忙吗?干杯:)

3

合并