build: begin using flake-parts feautures more and also compress lockfile

[DockterTeagle]

as mentioned in the discussion posts this is to use more of flake-parts and also compress lockfile, feel free to ignore or close.

[github-actions]

Review Checklist

Does this PR follow the Contribution Guidelines? Following is a partial checklist:

Proper conventional commit scoping:

• For example, fix(lsp): some lsp-related bugfix

• Pull request title has the appropriate conventional commit prefix.

If applicable:

• Tested
  • Tests have been added.
  • Tested manually (Steps to reproduce in PR description).
• Updated documentation.

[DockterTeagle]

why did you call it pre-commit check if it isnt applied at pre-commit? also I forgot to check, and for some reason changing to the new URL breaks lua checks?

[DockterTeagle]

where is pkgs.nvim-nightly-tests defined? I am trying to make sense of the devshell and to also figure out why tests are failing

[DockterTeagle]

so one of a few things happened,
A. these tests have been failing all the time, but are only now showing up(they are warnings)
B. cachix changed the lib to make it not the same(I cant find pre-commit-hooks.nix anywhere)
C. I have entered psychosis from lack of sleep, and
D. I am somehow messing up the liib

[DockterTeagle]

after further reflection I think that the lsp or the luarc got updated win regards to neotest and code_Action causing these warnings

[mrcjkb]

why did you call it pre-commit check if it isnt applied at pre-commit? also I forgot to check, and for some reason changing to the new URL breaks lua checks?

It is, in the devShell. The repo was renamed to git-hooks.nix though.

[DockterTeagle]

sorry misclick not fully ready

[mrcjkb]

I'll have to look at this at a later time.

[DockterTeagle]

fair enough, this would be my first time contributing to anything of note, so I want it to be correct

[mrcjkb]

Dead code?

[DockterTeagle]

overlayAttrs is trying to use the easy overlay provided by flake-parts

[DockterTeagle]

I left it commented as a note to use it later, if you are willing to, and also I dont think that it is that much of an improvement.

[mrcjkb]

after further reflection I think that the lsp or the luarc got updated win regards to neotest and code_Action causing these warnings

Yep, that happens sometimes.
Often, it's enough to add a ---@diagnostic disable-next-line comment (if it's not an actual type check error that was missed by previous lua-ls versions).

[mrcjkb]

This is not a feature. The correct commit prefix would be build:.

[DockterTeagle]

ah, thought feat meant like well a feat, but you are right. will rename later after thermo final in approximately 5 hours