Skip to content

Add Hol1kgmg repository - #1060

Merged
Pandapip1 merged 1 commit into
nix-community:mainfrom
Hol1kgmg:main
Sep 5, 2026
Merged

Add Hol1kgmg repository#1060
Pandapip1 merged 1 commit into
nix-community:mainfrom
Hol1kgmg:main

Conversation

@Hol1kgmg

@Hol1kgmg Hol1kgmg commented Jan 15, 2026

Copy link
Copy Markdown
Contributor

The following points apply when adding a new repository to repos.json

  • I ran ./bin/nur format-manifest after updating repos.json (We will use the same script in github actions to make sure we keep the format consistent)
  • By including this repository in NUR, I confirm that any copyrightable content in the repository (other than built derivations or patches, if applicable) is licensed under the MIT license
  • I confirm that meta.license and meta.sourceProvenance have been set correctly for any derivations for unfree or not built from source packages

Additionally, the following points are recommended:

  • All applicable meta fields have been filled out. See https://nixos.org/manual/nixpkgs/stable/#sec-standard-meta-attributes for more information. The following fields are particularly helpful and can always be filled out:
    • meta.description, so consumers can confirm that that your package is what they're looking for
    • meta.license, even for free packages
    • meta.homepage, for tracking and deduplication
    • meta.mainProgram, so that nix run works correctly

I had requested a review of this PR with GitHub Actions failing. I have made revisions, so please review it again.

@Hol1kgmg Hol1kgmg changed the title Add new repository entry for Hol1kgmg in repos.json Add Hol1kgmg repository Jan 15, 2026

@Pandapip1 Pandapip1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It doesn't look like any packages have been added.

@Hol1kgmg

Hol1kgmg commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

Apologies for the accidental close just now (reopened), and for the long delay overall.

@Pandapip1 addressing your earlier review comment — packages (herdr, yaskkserv2) have now been added to the repository, so it should no longer appear empty.

To recap the history: the original entry was merged in #1053 but reverted in #1055 due to an eval failure. I've since fixed the repository (verified with a full nix-env -f . -qa --json --meta evaluation completing without errors) and rebased this branch onto the latest main to resolve the merge conflict.

This should be ready for review again — thank you for your patience.

@Hol1kgmg Hol1kgmg reopened this Sep 5, 2026
@Hol1kgmg
Hol1kgmg requested a review from Pandapip1 September 5, 2026 12:23
@Pandapip1
Pandapip1 added this pull request to the merge queue Sep 5, 2026
Merged via the queue into nix-community:main with commit d46b3df Sep 5, 2026
4 checks passed
github-actions Bot pushed a commit to milahu/NUR that referenced this pull request Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants