Skip to content

fix: corrected variable names in validate_migration role [skip ci]#26

Merged
burigolucas merged 1 commit into
redhat-cop:mainfrom
sabre1041:validate_migration_fix_variables
May 1, 2026
Merged

fix: corrected variable names in validate_migration role [skip ci]#26
burigolucas merged 1 commit into
redhat-cop:mainfrom
sabre1041:validate_migration_fix_variables

Conversation

@sabre1041
Copy link
Copy Markdown
Contributor

@sabre1041 sabre1041 commented Apr 13, 2026

Description

Corrected variable names that were made during prior variable refactoring efforts

Discovered while reviewing #17

Type of Change

  • feat: A new feature
  • fix: A bug fix
  • docs: Documentation changes
  • style: Formatting, missing semi colons, etc; no code change
  • refactor: Refactoring production code
  • test: Adding missing tests, refactoring tests; no production code change
  • chore: Updating configs, etc; no production code change

Pull Request Title

Note: Please ensure your PR title follows Conventional Commits.
Example: feat: add role

  • I have formatted my PR title correctly.
  • I have added [skip ci] to the title (Maintainers only, if no release is needed).

Signed-off-by: Andrew Block <andy.block@gmail.com>
@sabre1041 sabre1041 changed the title fix: corrected variable names in validate_migration role fix: corrected variable names in validate_migration role [skip ci] Apr 14, 2026
Copy link
Copy Markdown

@spyrexd spyrexd left a comment

Choose a reason for hiding this comment

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

LGTM

@burigolucas burigolucas merged commit d20f0cc into redhat-cop:main May 1, 2026
48 of 51 checks passed
This was referenced May 6, 2026
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