Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Possible refactoring: cmd/nerdctl is very crowded? #3375

Closed
apostasie opened this issue Aug 28, 2024 · 2 comments
Closed

Possible refactoring: cmd/nerdctl is very crowded? #3375

apostasie opened this issue Aug 28, 2024 · 2 comments
Labels
duplicate This issue or pull request already exists

Comments

@apostasie
Copy link
Contributor

What is the problem you're trying to solve

The cmd/nerdctl folder is very crowded.

I find it hard to reason about, or figure out what depends on what.

This is not a huge problem - this falls more into the "developer quality of life" category.

Would we be open to putting that stuff under sub-folders, by category? (container, image, network, volume, system, etc)

Describe the solution you'd like

na

Additional context

No response

@AkihiroSuda
Copy link
Member

Would we be open to putting that stuff under sub-folders, by category? (container, image, network, volume, system, etc)

IIRC we have tried this before and faced an issue about splitting *_test.go

@AkihiroSuda
Copy link
Member

We can continue the discussion in:

@AkihiroSuda AkihiroSuda closed this as not planned Won't fix, can't repro, duplicate, stale Aug 29, 2024
@AkihiroSuda AkihiroSuda added duplicate This issue or pull request already exists and removed kind/refactor labels Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants