Skip to content

Conversation

@deepakdinesh1123
Copy link
Contributor

No description provided.

@deepakdinesh1123
Copy link
Contributor Author

Some of the benchmarks that I have written are query-related and require some of the changes made in this PR so I will be adding them after this gets merged.

@smithdc1
Copy link
Member

smithdc1 commented Jun 1, 2022

Hey @deepakdinesh1123 -- I've rebased, applied black, isort and checked for flake8 errors. This will need to be done for the other PRs too, could you start to work on those?

@deepakdinesh1123
Copy link
Contributor Author

isort check is failing with an error but I am not getting any error on my machine. I tried to configure known_first_party and other options but to no avail.

@smithdc1
Copy link
Member

smithdc1 commented Jun 2, 2022

I'll have a look at boosting our developer experience and use pre-commit / pre-commit ci so lint errors get automatically corrected.

@smithdc1 smithdc1 merged commit 36d75b1 into django:main Jun 2, 2022
@deepakdinesh1123 deepakdinesh1123 deleted the query_multifield branch June 4, 2022 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants