Skip to content

Commit

Permalink
[backends] Remove obsolete TODOs
Browse files Browse the repository at this point in the history
  • Loading branch information
ricab committed Dec 13, 2023
1 parent 8561260 commit 71971da
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/platform/backends/shared/base_virtual_machine.h
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ inline int multipass::BaseVirtualMachine::get_snapshot_count() const
return snapshot_count;
}

inline void multipass::BaseVirtualMachine::require_snapshots_support() const // TODO@nomerge implement on supporting
inline void multipass::BaseVirtualMachine::require_snapshots_support() const

Check warning on line 146 in src/platform/backends/shared/base_virtual_machine.h

View check run for this annotation

Codecov / codecov/patch

src/platform/backends/shared/base_virtual_machine.h#L146

Added line #L146 was not covered by tests
{
throw NotImplementedOnThisBackendException{"Snapshots"};

Check warning on line 148 in src/platform/backends/shared/base_virtual_machine.h

View check run for this annotation

Codecov / codecov/patch

src/platform/backends/shared/base_virtual_machine.h#L148

Added line #L148 was not covered by tests
}
Expand Down
3 changes: 1 addition & 2 deletions src/platform/backends/shared/base_virtual_machine_factory.h
Original file line number Diff line number Diff line change
Expand Up @@ -104,8 +104,7 @@ inline void multipass::BaseVirtualMachineFactory::remove_resources_for(const std
instance_dir.removeRecursively();
}

inline void
multipass::BaseVirtualMachineFactory::require_snapshots_support() const // TODO@nomerge override where supported
inline void multipass::BaseVirtualMachineFactory::require_snapshots_support() const

Check warning on line 107 in src/platform/backends/shared/base_virtual_machine_factory.h

View check run for this annotation

Codecov / codecov/patch

src/platform/backends/shared/base_virtual_machine_factory.h#L107

Added line #L107 was not covered by tests
{
throw NotImplementedOnThisBackendException{"Snapshots"};

Check warning on line 109 in src/platform/backends/shared/base_virtual_machine_factory.h

View check run for this annotation

Codecov / codecov/patch

src/platform/backends/shared/base_virtual_machine_factory.h#L109

Added line #L109 was not covered by tests
}
Expand Down

0 comments on commit 71971da

Please sign in to comment.