sanic-org / sanic

Accelerate your web app development | Build fast. Run fast.

Home Page:https://sanic.dev

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

remove tracerite from the standard install

TimurDela opened this issue · comments

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

Tracerite raises many deprecation warnings, I am not developing in a Notebook environment and I feel that I should not have to install tracerite in order to use sanic.
These deprecation warnings make the development experience with Sanic much less pleasant (for instance my unit tests reports get filled with those warnings).

Would it be possible to remove tracerite from the sanic dependencies?

Code snippet

No response

Expected Behavior

No response

How do you run Sanic?

As a module

Operating System

MacOS

Sanic Version

23.6.0

Additional context

No response

Recategorized appropriately to a deprecation, as deprecation warnings are not bugs.

For discussion: @sanic-org/core-devs should tracerite be moved to sanic-ext as a dependency rather than part of sanic itself?

I believe it's a dependency here because we use it for custom error pages. @Tronic and @ahopkins might know more about it. However, we also maintain tracerite so we should be able to fix those warnings, hopefully.

The problematic module that Tracerite used has been removed. Closing this in favor of keeping discussion to sanic-org/tracerite#8

Note that the tracerite and html5tagger modules are lightweight and don't bring in any other dependencies (such as notebooks). Tracerite is an integral part of Sanic exception handling.