-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
spectral-language-server: init at 1.0.8 #301730
Conversation
Result of 1 package built:
|
@arduano @natsukium Would you review this pull request? 🙇 |
Anyway the rest lgtm, but I'm new here, so I'll let more experienced contributors judge. Welcome to nixpkgs btw |
Thank you for your review! I'll be waiting for their judgement too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution!
I left some comments, could you take a look at them?
67e6513
to
dfc7069
Compare
@ofborg eval |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Besides this, LGTM.
dfc7069
to
652444b
Compare
Co-authored-by: OTABI Tomoya <[email protected]>
652444b
to
ef0d7b7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for your hard work.
Welcome to nixpkgs!🎉
close #283363
Description of changes
spectral-language-server is a language server that includes Spectral, a flexible object linter with out-of-the-box support for OpenAPI v2 and v3, JSON Schema, and AsyncAPI.
I defined a derivation because #283363 requests this package.
https://github.com/luizcorreia/spectral-language-server
As this is my first time writing code on nixpkgs, I would be grateful if you could tell me if there are any bugs.
In particular, I'm wondering if my workaround for @rollup/pluginutils not providing type definitions for NodeNext is the right one.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.