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

support ruamel codemod #6

Merged
merged 1 commit into from
Aug 28, 2023
Merged

support ruamel codemod #6

merged 1 commit into from
Aug 28, 2023

Conversation

clavedeluna
Copy link
Contributor

@clavedeluna clavedeluna commented Aug 25, 2023

Overview

New codemod to support ruamel

Description

  • ruamel is a relatively well known yaml api, so adding support for ensuring its safe use
  • to try to move toward the new api idea, I made update_arg_target in hopes we can generalize it

Documentation

Describe your testing approach

  • similar as other codemods

Performance

  • How do you expect this PR to affect performance?

Additional Details

  • any follow up tickets or discussion
  • Any specific merge / deploy details

return updated_node


def update_arg_target(original_args, target_arg):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️

@clavedeluna clavedeluna merged commit d2e4968 into main Aug 28, 2023
6 checks passed
@clavedeluna clavedeluna deleted the ruamel-codemod branch August 28, 2023 11:11
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