We can only land this once all migrations in bug 1970521 are completed. Since it's not possible to use `orjson` in all cases, we'll also need to add a way to suppress this lint via pragma so that in cases where it's necessary to use the standard library `json` the linter won't complain. ``` # noqa necessary stdlib json import ``` Or something similar to the above (I'm not sure what the standard is for `noqa`). The linter output should also communicate the existence of this pragma and when to use it, since most people getting this linter error will have none of this context. Adding the linter should be one patch, then adding the pragmas to all the imports that could not be migrated to `orson` should be another.
Bug 1973586 Comment 0 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
We can only land this once all migrations in bug 1970521 are completed. Since it's not possible to use `orjson` in all cases, we'll also need to add a way to suppress this lint via pragma so that in cases where it's necessary to use the standard library `json` the linter won't complain. ``` # noqa: TID251 ``` Or something similar to the above (I'm not sure what the standard is for `noqa`). The linter output should also communicate the existence of this pragma and when to use it, since most people getting this linter error will have none of this context. Adding the linter should be one patch, then adding the pragmas to all the imports that could not be migrated to `orson` should be another.
We can only land this once all migrations in bug 1970521 are completed. Since it's not possible to use `orjson` in all cases, we'll also need to add a way to suppress this lint via pragma so that in cases where it's necessary to use the standard library `json` the linter won't complain. ``` # noqa: TID251 ``` ~~Or something similar to the above (I'm not sure what the standard is for `noqa`).~~ The linter output should also communicate the existence of this pragma and when to use it, since most people getting this linter error will have none of this context. Adding the linter should be one patch, then adding the pragmas to all the imports that could not be migrated to `orson` should be another.