Skip to content
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

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

apodhrad
Copy link
Contributor

@apodhrad apodhrad commented Oct 4, 2023

No description provided.

@apodhrad apodhrad force-pushed the aws-resource-explorer branch from 7a9f647 to cb9dbea Compare October 13, 2023 16:27
@ogajduse
Copy link
Member

pre-commit.ci autofix

Copy link
Contributor

@jyejare jyejare left a 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!

wrapanapi/systems/ec2.py Outdated Show resolved Hide resolved
wrapanapi/systems/ec2.py Outdated Show resolved Hide resolved
wrapanapi/systems/ec2.py Outdated Show resolved Hide resolved
wrapanapi/systems/ec2.py Outdated Show resolved Hide resolved
wrapanapi/systems/ec2.py Outdated Show resolved Hide resolved
wrapanapi/systems/ec2.py Outdated Show resolved Hide resolved
wrapanapi/systems/ec2.py Outdated Show resolved Hide resolved
@apodhrad apodhrad force-pushed the aws-resource-explorer branch from bfafd67 to f9467d8 Compare October 23, 2023 08:01
@apodhrad
Copy link
Contributor Author

Hi @jyejare thanks for the review. I have applied all your suggestions.

@ogajduse
Copy link
Member

ogajduse commented Oct 23, 2023

@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 pr€-commit.ci autofix on this PR as I did above to ask pre-commit.ci to fix it.

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.

@apodhrad apodhrad force-pushed the aws-resource-explorer branch from f9467d8 to 3609867 Compare October 23, 2023 14:44
Signed-off-by: Andrej Podhradsky <[email protected]>
@apodhrad apodhrad force-pushed the aws-resource-explorer branch from 3609867 to 39304b6 Compare October 23, 2023 14:59
@ogajduse ogajduse requested a review from jyejare October 23, 2023 15:18
@jyejare jyejare merged commit c3e734f into RedHatQE:master Oct 25, 2023
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.

3 participants