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

s3tests_boto3/functional/test_sts.py: clean up, fix pep warnings #586

Open
wants to merge 42 commits into
base: master
Choose a base branch
from

Conversation

rkhudov
Copy link

@rkhudov rkhudov commented Sep 9, 2024

No functional changes.

  • Remove unused imports
  • Add whitespaces for variable assignment
  • Remove extra whitespaces for arguments assignment
  • Remove unused variables

…x too long lines, remove redundant brackets

Signed-off-by: Rostyslav Khudov <[email protected]>
…, fix too long lines, remove redundant brackets

Signed-off-by: Rostyslav Khudov <[email protected]>
…, fix too long lines, remove redundant brackets

Signed-off-by: Rostyslav Khudov <[email protected]>
…dd whitespaces, remove redundant variable assignment

Signed-off-by: Rostyslav Khudov <[email protected]>
…paces, remove redundant variable assignment

Signed-off-by: Rostyslav Khudov <[email protected]>
… add whitespaces, remove redundant variable assignment

Signed-off-by: Rostyslav Khudov <[email protected]>
…espaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…t_creds_denied add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…espaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…spaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…dd whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…xistent add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…existent add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…fferent_buckets add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…me_bucket add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…bj_denial add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ession_policy add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…rent_op_permissions add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…deny_effect add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…deny_on_same_op add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…cy_role_arn add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…cy_session_arn add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
… add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ole_policy add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…cy_deny add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_with_sub add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_with_azp add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_with_request_tag add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_with_principal_tag add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_for_all_values add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_for_all_values_deny add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_tag_keys_trust_policy add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_tag_keys_role_policy add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_resource_tag add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_resource_tag_deny add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_wrong_resource_tag_deny add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_resource_tag_princ_tag add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_resource_tag_copy_obj add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
…ity_resource_tag add whitespaces, use correct ident level, remove unusued variables

Signed-off-by: Rostyslav Khudov <[email protected]>
@rkhudov
Copy link
Author

rkhudov commented Oct 2, 2024

@cbodley Hello, will it get chance to be reviewed?

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.

1 participant