-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
Jetbrains plugins #191341
Jetbrains plugins #191341
Conversation
c19809a
to
8b02715
Compare
1f31c8b
to
79848b6
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.
I'm have successfully used this PR to add GitHub Copilot to IntelliJ and Clion in my own setup. Thanks!
It is a bit unwieldy to use the functions here though, maybe an interface like this would be more ergonomic:
with pkgs.jetbrains;
let
copilot = mkJetbrainsPlugin {
src = fetchJetbrainsPlugin {
id = "1234";
hash = "....";
};
buildInputs = with pkgs; [ patchelf gcc-unwrapped ];
build = ''
# ... additional build steps
'';
};
in idea-ultimate.withPlugins [ copilot ]
Also, instead of adding exceptions for specialPlugins
, we could provide a collection of plugins (pkgs.jetbrains.plugins.copilot
for example).
What do you think?
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/clion-github-copilot-fails-to-login/22197/2 |
How would that interface get the hash of the api response? Would there be a way to make nix ignore impure derivations if its only dependency is content addressed? As long as updating plugins/adding new plugins isn't too hard, I think the plugin list would work. |
Hi @GenericNerdyUsername and @rien, thank you for the PR and copilot code!! I've applied it already with overlay based on:
To install GitHub copilot for idea-ultimate and it works like a charm! :) Here is the overlay: ktor/nixos@35f4071 |
79848b6
to
579d420
Compare
I think this is usually done by creating an |
ac72005
to
3bc9388
Compare
I've added the update script and reworked how you add plugins, though I'm really tired and have definitely missed something, I'm just putting this out there so other people can suggest improvements or use it |
3bc9388
to
9bb586c
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/1402 |
9bb586c
to
751107a
Compare
9b32d54
to
adcfa5f
Compare
446339f
to
39bab6e
Compare
Whats stopping this from getting merged? |
Thank you for your effort! Everything works as expected on my x86_64 laptop. |
39bab6e
to
9cbb49a
Compare
maintainers don't seem to be merging jetbrains related PRs, all of them seems to have been stalled for a while now. furthermore darwin users aren't really reviewing or helping to diagnose build issues |
9cbb49a
to
2ceb8b5
Compare
+1 on this. I'd love to not have to use a 3rd party flake to be able to use Github Copilot. |
571334c
to
27eb514
Compare
27eb514
to
7f673bc
Compare
I made a new version (#223593) which checks for compatibility. |
Description of changes
This commit adds support for declarative plugins (including ones which need patching) and some helper functions for fetching them from https://plugins.jetbrains.com. Untested on darwin. Closes #189183 .
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes