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
vimPlugins.defx-nvim: init at 2019-08-19 #67160
Conversation
Why is the commit message saying
|
The github repo for Defx shows a 1.0 tagged release. However I'm not sure if that's what nixpkgs looks for when building derivations for vim plugins. |
Since vim plugins usually don't have releases and are intended to be shipped straight from master, the nixpkgs vim infrastructure doesn't look from releases but instead takes the latest master commit. If you want to use versions instead, you have to side-step the automated infrastructure. Please try to find out (or ask) if this plugin is intended to be used from master or from the tagged version. |
I've tried getting in contact with Shougo multiple times through email now, but I can't get ahold of them. However, after confirming in a different distro how plugin managers handle it, they all pull from master, rather than one of the releases. What should I do here regarding the commit message? |
When we track master instead of releases, we usually use the date of the latest commit as the version. So you need to find out when the latest commit on master was made and then amend the commit to |
237d3f0
to
fd90f36
Compare
Is this ok now? I've rebased to an updated master, amended my last commit to fix the init message, and force-pushed the result to my current branch |
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.
Yes, looks good now. Thank you!
Motivation for this change
Things done
Comparing my changes to similar plugin init PRs, I see I somehow got a lot more automatic changes applied to
generated.nix
. I prepared this following the steps outlined in here. Is this correct?sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @