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

Remove unused files #743

Open
Neurone opened this issue Dec 7, 2023 · 0 comments
Open

Remove unused files #743

Neurone opened this issue Dec 7, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@Neurone
Copy link
Contributor

Neurone commented Dec 7, 2023

Problem

I see comments in flake.nix file that this file, and the related flake.lock I assume, are there just to be used by Skyler Ross [email protected]`; this can be confusing and misleading. As a dev, I prefer not having useless files on my repo, or assuming I can use Nix because I see the .nix file, and than I discover it's not there for me.

todo-list.txt seems outdated (last update Dec 2022). It's confusing if I should still look at it or not to understand the current SDK features.

Solution

If those files are there for Skyler only, we should remove them. Skyler can keep those files for itself ;)
If those files are needed in some custom dev env, we can add those files to .gitignore, the same we do with .idea or .task

If, on the contrary, those files can be helpful as an optional tool, because they want to use NixOS, I suggest to update them, keep them updated, and remove the comment.

About the todo-list, keep it up to date or remove it to avoid confusion.

Alternatives

No response

@Neurone Neurone added the enhancement New feature or request label Dec 7, 2023
@SimiHunjan SimiHunjan added this to the v0.24.0 milestone Jan 3, 2024
@SimiHunjan SimiHunjan removed this from the v0.24.0 milestone Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants