Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

acking the 4.18 and 4.19 issues #81

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

jtaleric
Copy link
Member

@jtaleric jtaleric commented Dec 17, 2024

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

Description

Acking the current issues we have identified in 4.18 and 4.19.

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please describe the System Under Test.
  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

Acking the current issues we have identified in 4.18 and 4.19.

Signed-off-by: Joe Talerico aka rook <joe.talerico@gmail.com>
@jtaleric
Copy link
Member Author

weird, the pylint seems to be using an old commit 😕

@josecastillolema
Copy link
Collaborator

/lgtm

Copy link
Collaborator

@vishnuchalla vishnuchalla left a comment

Choose a reason for hiding this comment

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

/lgtm

@josecastillolema josecastillolema merged commit 3584d8f into cloud-bulldozer:main Dec 18, 2024
4 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants