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

Now sees attribute set arguments as regular attribute sets and throws ; expected warnings #325

Closed
BirdeeHub opened this issue Jan 26, 2024 · 12 comments
Labels
question Generally ask questions

Comments

@BirdeeHub
Copy link

BirdeeHub commented Jan 26, 2024

Now sees attribute set arguments as regular attribute sets and throws = and ; expected warnings
Something in the last couple days broke it. I really have no idea what but it looks like this

Screenshot_2024-01-25_21-27-04

Edit:

the last working commit Ive found so far is 5b33874

The commit immediately after it, ccf9815 actually makes the lsp client crash completely

@inclyc
Copy link
Member

inclyc commented Jan 26, 2024

Now sees attribute set arguments as regular attribute sets and throws = and ; expected warnings Something in the last couple days broke it. I really have no idea what but it looks like this

Screenshot_2024-01-25_21-27-04

Edit:

the last working commit Ive found so far is 5b33874

The commit immediately after it, ccf9815 actually makes the lsp client crash completely

Thanks for contribution!

Looks like you are on the unstable nixd-next branch, as noted in README.md1, it is on an early stage and working in progress. Pictures shown here are actually lambdas, which may not yet implemented. See #283 for the progress.

Footnotes

  1. WIP note in README.md

@inclyc
Copy link
Member

inclyc commented Jan 26, 2024

Are you using the overlay or something related to default branch in this repo? We have stable releases in nixpkgs(pkgs.nixd) perhaps you can use that package.

@inclyc
Copy link
Member

inclyc commented Jan 26, 2024

I just set the default branch back to main, to avoid such issue...

@BirdeeHub
Copy link
Author

Im using the overlay. I have it pinned now to the last working version in my nix config. Is the newest version not supported or something? If this branch is a work in progress still I will leave it pinned I suppose?

@BirdeeHub
Copy link
Author

BirdeeHub commented Jan 26, 2024

https://github.com/BirdeeHub/nixCats-nvim/tree/nixCats-5.0.0

Im using this (because I made it and its super cool), although it seems to happen with the nixos and home manager modules as well.

@inclyc
Copy link
Member

inclyc commented Jan 26, 2024

Im using the overlay. I have it pinned now to the last working version in my nix config. Is the newest version not supported or something? If this branch is a work in progress still I will leave it pinned I suppose?

Maybe just use the latest stable release. Normally main branch is always working and has no breaking changes, however nixd-next is a big refactor (nearly a complete rewrite with complete new parser, semantic analysis and architecture), and it has many commits ahead main branch.

@BirdeeHub
Copy link
Author

hmmm okok. I was unaware. Well, FYI then, nixd next has broken attribute set arguments XD

@inclyc
Copy link
Member

inclyc commented Jan 26, 2024

Im using the overlay. I have it pinned now to the last working version in my nix config. Is the newest version not supported or something? If this branch is a work in progress still I will leave it pinned I suppose?

I would like suggest completely not using nixd flake overlay, if you are not developers. Just use nixpkgs version, that's also maintained by me.

@inclyc
Copy link
Member

inclyc commented Jan 26, 2024

hmmm okok. I was unaware. Well, FYI then, nixd next has broken attribute set arguments XD

That's not "broken", just completely NOT implemented yet. :(

@BirdeeHub
Copy link
Author

Im using the overlay. I have it pinned now to the last working version in my nix config. Is the newest version not supported or something? If this branch is a work in progress still I will leave it pinned I suppose?

I would like suggest completely not using nixd flake overlay, if you are not developers. Just use nixpkgs version, that's also maintained by me.

Somehow, when I was first looking for nixd on nixpkgs I was unable to find it?! IDK how I found it just fine this time....

This is good news that I can remove that flake input from nixCats-nvim

@inclyc
Copy link
Member

inclyc commented Jan 26, 2024

image

These features are not supported yet, and may work a few days later.

@BirdeeHub
Copy link
Author

Do I close this issue then? leave it open? glad I found out I was using the wrong version I guess

@inclyc inclyc added the question Generally ask questions label Jan 26, 2024
@inclyc inclyc closed this as completed Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Generally ask questions
Projects
None yet
Development

No branches or pull requests

2 participants