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

[fix] add move-heirarchy label to LMT #435

Merged
merged 1 commit into from
Jul 31, 2024
Merged

[fix] add move-heirarchy label to LMT #435

merged 1 commit into from
Jul 31, 2024

Conversation

AshleyDumaine
Copy link
Contributor

What this PR does / why we need it: This makes it so LinodeMachineTemplates aren't left behind after a clusterctl move operation.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

TODOs:

  • squashed commits
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

Copy link

codecov bot commented Jul 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.09%. Comparing base (062d08a) to head (a087677).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #435   +/-   ##
=======================================
  Coverage   65.09%   65.09%           
=======================================
  Files          68       68           
  Lines        3521     3521           
=======================================
  Hits         2292     2292           
  Misses       1051     1051           
  Partials      178      178           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AshleyDumaine AshleyDumaine merged commit d0d96a7 into main Jul 31, 2024
12 checks passed
@AshleyDumaine AshleyDumaine deleted the move-lmt branch July 31, 2024 18:02
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants