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

vimPlugins.lze: 0.8.1 -> 0.9.1 #391813

Merged
merged 1 commit into from
Mar 21, 2025
Merged

Conversation

BirdeeHub
Copy link
Contributor

added ability for handler add and modify hooks to defer a function for later

@GaetanLepage @teto Do either of you know the process for requesting a new invite to the github org for nixpkgs so that I can request review and stuff? I missed my invite long ago and now am aware that Im missing out.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@GaetanLepage
Copy link
Contributor

@GaetanLepage @teto Do either of you know the process for requesting a new invite to the github org for nixpkgs so that I can request review and stuff? I missed my invite long ago and now am aware that Im missing out.

I had no idea that being in the org was needed to request reviews...
Can you request to join from this web page? https://github.com/orgs/NixOS/people

@BirdeeHub
Copy link
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 391813


x86_64-linux

✅ 9 packages built:
  • lua51Packages.lze
  • lua51Packages.lzextras
  • luaPackages.lze (lua52Packages.lze)
  • lua53Packages.lze
  • lua54Packages.lze
  • luajitPackages.lze
  • luajitPackages.lzextras
  • vimPlugins.lze
  • vimPlugins.lzextras

@BirdeeHub
Copy link
Contributor Author

BirdeeHub commented Mar 21, 2025

@GaetanLepage @teto Do either of you know the process for requesting a new invite to the github org for nixpkgs so that I can request review and stuff? I missed my invite long ago and now am aware that Im missing out.

I had no idea that being in the org was needed to request reviews... Can you request to join from this web page? https://github.com/orgs/NixOS/people

unfortunately no, I can only follow people from that page

@BirdeeHub
Copy link
Contributor Author

BirdeeHub commented Mar 21, 2025

It sends out an auto invite on your first contribution, but I missed that over a year ago and didnt notice

@BirdeeHub
Copy link
Contributor Author

BirdeeHub commented Mar 21, 2025

but yeah thats why I have been pinging people with @ and being confused basically, which I didnt know until recently

@BirdeeHub
Copy link
Contributor Author

Ive heard about a matrix channel as well and was told to go there by someone and try to ask, but I dont know where that is, I havent tried making a post on discourse about it yet either.

@BirdeeHub
Copy link
Contributor Author

also hang on, I might have 1 more update so dont merge this yet, I will run nixpkgs review again when ready

@GaetanLepage
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 391813


x86_64-linux

✅ 9 packages built:
  • lua51Packages.lze
  • lua51Packages.lzextras
  • luaPackages.lze (lua52Packages.lze)
  • lua53Packages.lze
  • lua54Packages.lze
  • luajitPackages.lze
  • luajitPackages.lzextras
  • vimPlugins.lze
  • vimPlugins.lzextras

aarch64-linux

✅ 9 packages built:
  • lua51Packages.lze
  • lua51Packages.lzextras
  • luaPackages.lze (lua52Packages.lze)
  • lua53Packages.lze
  • lua54Packages.lze
  • luajitPackages.lze
  • luajitPackages.lzextras
  • vimPlugins.lze
  • vimPlugins.lzextras

x86_64-darwin

✅ 9 packages built:
  • lua51Packages.lze
  • lua51Packages.lzextras
  • luaPackages.lze (lua52Packages.lze)
  • lua53Packages.lze
  • lua54Packages.lze
  • luajitPackages.lze
  • luajitPackages.lzextras
  • vimPlugins.lze
  • vimPlugins.lzextras

aarch64-darwin

✅ 9 packages built:
  • lua51Packages.lze
  • lua51Packages.lzextras
  • luaPackages.lze (lua52Packages.lze)
  • lua53Packages.lze
  • lua54Packages.lze
  • luajitPackages.lze
  • luajitPackages.lzextras
  • vimPlugins.lze
  • vimPlugins.lzextras

@BirdeeHub BirdeeHub marked this pull request as draft March 21, 2025 11:27
@BirdeeHub
Copy link
Contributor Author

yeah I have a fix to where I want these deferred functions to run 1s

@BirdeeHub BirdeeHub marked this pull request as ready for review March 21, 2025 11:32
@BirdeeHub
Copy link
Contributor Author

better.

Sorry, realized beforeAll was no longer beforeAll

@BirdeeHub
Copy link
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 391813


x86_64-linux

✅ 9 packages built:
  • lua51Packages.lze
  • lua51Packages.lzextras
  • luaPackages.lze (lua52Packages.lze)
  • lua53Packages.lze
  • lua54Packages.lze
  • luajitPackages.lze
  • luajitPackages.lzextras
  • vimPlugins.lze
  • vimPlugins.lzextras

@BirdeeHub
Copy link
Contributor Author

ok now ready :)

@BirdeeHub BirdeeHub changed the title vimPlugins.lze: 0.8.1 -> 0.9.0 vimPlugins.lze: 0.8.1 -> 0.9.1 Mar 21, 2025
@GaetanLepage GaetanLepage merged commit de14322 into NixOS:master Mar 21, 2025
44 of 47 checks passed
@BirdeeHub BirdeeHub deleted the vimPlugins.lze branch March 21, 2025 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants