-
Notifications
You must be signed in to change notification settings - Fork 83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for AWS Resource Explorer #470
Conversation
7a9f647
to
cb9dbea
Compare
pre-commit.ci autofix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK pending some minor comments!
bfafd67
to
f9467d8
Compare
Hi @jyejare thanks for the review. I have applied all your suggestions. |
@apodhrad Please make sure that the pre-commit checks are passing. I see that black is complaining about bad formatting. If do not plan any further changes, you can simply comment Edit: Also some of the unit tests failed to collect. Can you please take a look at the failures too? They seem to be related to the typing you added. |
f9467d8
to
3609867
Compare
Signed-off-by: Andrej Podhradsky <[email protected]>
3609867
to
39304b6
Compare
No description provided.