Bibo-Joshi · GitHub

@Bibo-Joshi

Copy link Copy Markdown

Member

@Bibo-Joshi

@Bibo-Joshi

github-actions[bot]

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 :)

tsnoam

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.

harshil21

@Bibo-Joshi

@Bibo-Joshi

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

Read the original on github.com ↗