Skip to content

Conversation

@alexandear
Copy link
Member

The PR refactors code in the pkg/cidata:

  • rename y to instConfig;
  • pass only necessary params to the setupEnv function.

Signed-off-by: Oleksandr Redko <oleksandr.red+github@gmail.com>
Copy link
Member

@jandubois jandubois left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

Given recent changes I think the variable should now be called instCfg, but that should probably be done globally in a separate PR.

@jandubois
Copy link
Member

jandubois commented Sep 30, 2024

Given recent changes I think the variable should now be called instCfg, but that should probably be done globally in a separate PR.

Hmm, there are not that many instance of instConfig or inst.Config, so maybe it can be included in this PR?

$ grep -rc -E 'inst\.?Config' cmd pkg | grep -v :0
cmd/limactl/shell.go:5
cmd/limactl/show-ssh.go:4
pkg/hostagent/hostagent.go:36
pkg/hostagent/inotify.go:1
pkg/hostagent/mount.go:1
pkg/hostagent/requirements.go:5
pkg/instance/ansible.go:1
pkg/instance/create.go:6
pkg/instance/start.go:2
pkg/store/instance.go:1

@jandubois jandubois merged commit 2d7ed91 into lima-vm:master Sep 30, 2024
@jandubois jandubois added this to the v1.0 milestone Sep 30, 2024
@jandubois jandubois added the kind/refactoring Refactoring label Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactoring Refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants