Это было бы достаточно просто сделать, но это было бы серьезным изменением.

AxiosError должен наследоваться от Error. Это не нарушило бы изменения.

Было бы, если кто-то проверяет error.constructor.name === 'Error'

Я согласен, но это незначительное изменение. Можем ли мы пожертвовать этим ради лучшего API?

Я за это, но лично я бы предпочел, если сопровождающие, конечно, согласятся, чтобы мы заменили axios на semver, v1.0.0.

Axios, безусловно, кажется достаточно стабильным для этого, а следование semver-major предотвращает поломку.

С другой стороны, учитывая широкое использование axios и предпочтительную HTTP-библиотеку React и Vue.js, критическое изменение может иметь серьезные последствия, которые необходимо учитывать.

На самом деле, теперь, когда я проверил, axios работает над веткой v1.0.0, я мог бы открыть PR для этого сегодня.

@mzabriskie, @nickuraltsev

@ДжастинБеквит

/cc @Khaledgarbaya @axe312ger @zcei

О, да, это очень помогло бы. Такая же проблема в нашем SDK.

Можем ли мы добавить это в #1333 @Khaledgarbaya?

PR уже есть, я добавил мягкое исправление для версий 0.x в виде флага в #1419 и изменение типа ошибки для 1.x в #1420, см. связанные PR

@axe312ger @Khaledgarbaya есть новости по этому поводу?

@ axe312ger какие-нибудь обновления, пожалуйста?

Придется передать @Khaledgarbaya 😅

Это было бы замечательно. Сейчас столкнулся с этой проблемой!

Есть ли прогресс в этом вопросе? Полезно отличать ошибки axios от других ошибок. И добавление флага isAxiosError к ошибкам, выдаваемым axios https://github.com/axios/axios/pull/1419 , является хорошим решением без нарушения изменений. Рассмотрите возможность добавления этого флага в версию патча, пожалуйста?

+1

@Khaledgarbaya какие-нибудь обновления по этому поводу?

Мне все еще нравится это, так как у всех нас есть работа, не стесняйтесь вносить свой вклад в качестве PR

@axe312ger У нас уже 2 PR за 5 месяцев!

1420 и № 1419

Упс. Трудно следить за этим репо.

Я бы предпочел, чтобы https://github.com/axios/axios/pull/1419 оказался в master, так как переписывание v1 в настоящее время заморожено из-за нехватки времени на нашей стороне.

Можете ли вы разрешить конфликты и упомянуть некоторых людей там?

Теперь это исправлено в https://github.com/axios/axios/pull/1419.

Это было быстро :)

Это был открытый код :)

Сила GitHub упоминает :trollface:

@axe312ger @Khaledgarbaya , не слишком ли рано просить об освобождении, пожалуйста?

@AyushG3112 К сожалению, у меня нет прав на выпуск npm

@nickuraltsev @emilyemorehouse есть ли планы выпустить это в ближайшее время?

Думаю, если это реализовать, то будет практичнее и понятнее, чем использование.

@axe312ger @Khaledgarbaya @nickuraltsev @emilyemorehouse кто -нибудь из вас может @-упомянуть кого-то с правами на публикацию npm?

AxiosError был бы лучше... Может кто-нибудь опубликовать изменение isAxiosError, пожалуйста?

Поскольку, по крайней мере, нам нужна какая-то функция, поддерживающая защиту типов, например function isAxiosError(error: Error): error is AxiosError { return !!error.isAxiosError } , возможно, было бы лучше иметь ее на стороне библиотеки.

@Khaledgarbaya @axe312ger Прошли месяцы, можно ли выпустить релиз, пожалуйста?

У нас нет прав публиковать аксиомы в npm

Не могли бы вы отметить кого-то, кто делает, пожалуйста?

Вы сделали это сами в декабре прошлого года https://github.com/axios/axios/issues/1415#issuecomment -449578470 - Ник и Эмили имеют права на публикацию.

👋 Я обрабатывал все выпуски для согласованности. Как только CI снова станет зеленым, я выпущу!