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

Remove unsed imports #892

Closed

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jan 27, 2022

Creating a fresh PR because the broken ansible/gate run from the old CI was blocking gating.

Was approved as #852 - no changes

SUMMARY

My local tests are flagging that we've picked up some unused imports again.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

aws_glue_job
cloudfront_info
rds_option_group_info

ADDITIONAL INFORMATION

@tremble tremble requested a review from markuman January 27, 2022 15:34
@markuman
Copy link
Member

recheck

@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request module module owner_pr PR created by owner/maintainer plugins plugin (any type) labels Jan 27, 2022
@goneri goneri added the mergeit Merge the PR (SoftwareFactory) label Jan 27, 2022
@softwarefactory-project-zuul
Copy link
Contributor

@softwarefactory-project-zuul
Copy link
Contributor

Pull request merge failed: Required status check "ansible/gate" is failing.

@goneri goneri closed this Jan 27, 2022
@goneri goneri reopened this Jan 27, 2022
@goneri
Copy link
Member

goneri commented Jan 27, 2022

@tremble the ansible/gate status from the old dashboard.zuul.ansible.com is now disabled and we cannot rerun it. I think it will be hard to cleam up the CI status from this PR. May I ask you to close this PR and push a new one?

@tremble tremble mentioned this pull request Jan 28, 2022
@tremble
Copy link
Contributor Author

tremble commented Jan 28, 2022

I initially tried recreating this one as a fresh PR from the same branch. Unfortunately, it looks like GitHub was being "clever", spotted the commits were the same and copied the pipelines over. #896 looks to have been more successful (fresh branch, fresh commit).

@tremble tremble closed this Jan 28, 2022
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jan 30, 2022
Cleanup unused imports

(#852 / #892) Let's try once more...
SUMMARY
My local tests are flagging that we've picked up some unused imports again.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_glue_job
cloudfront_info
rds_option_group_info
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
patchback bot pushed a commit that referenced this pull request Jan 30, 2022
Cleanup unused imports

(#852 / #892) Let's try once more...
SUMMARY
My local tests are flagging that we've picked up some unused imports again.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_glue_job
cloudfront_info
rds_option_group_info
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit 017081c)
tremble added a commit that referenced this pull request Jan 30, 2022
Cleanup unused imports

(#852 / #892) Let's try once more...
SUMMARY
My local tests are flagging that we've picked up some unused imports again.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_glue_job
cloudfront_info
rds_option_group_info
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit 017081c)

Co-authored-by: Mark Chappell <[email protected]>
@tremble tremble deleted the sanity/unused-202201 branch July 7, 2022 19:27
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…collections#892)

Bump from 5.0.0 to 5.0.0-dev0 in case of accidental release

SUMMARY
Bump from 5.0.0 to 5.0.0-dev0 in case of accidental release
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
galaxy.yml
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
Cleanup unused imports

(ansible-collections#852 / ansible-collections#892) Let's try once more...
SUMMARY
My local tests are flagging that we've picked up some unused imports again.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_glue_job
cloudfront_info
rds_option_group_info
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@017081c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request mergeit Merge the PR (SoftwareFactory) module module owner_pr PR created by owner/maintainer plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants