Copy link
Copy Markdown
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey! Looks like you edited the (optional) requirements or the pre-commit hooks. I'm just a friendly reminder to keep the additional dependencies for the hooks in sync with the requirements :)
Copy link
Copy Markdown
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm.
still, check the comments. you might want to address some of them.
Bibo-Joshi added a commit that referenced this pull request
Oct 23, 2023* Follow up for #3935 * Also adds tests that make sure that `ApplicationBuilder` has all necessary methods
Merged
4 tasks