Skip to content

tiny: add module#2735

Merged
berbiche merged 6 commits intonix-community:masterfrom
kmaasrud:tiny
Feb 26, 2022
Merged

tiny: add module#2735
berbiche merged 6 commits intonix-community:masterfrom
kmaasrud:tiny

Conversation

@kmaasrud
Copy link
Copy Markdown
Contributor

@kmaasrud kmaasrud commented Feb 19, 2022

Description

Added module for tiny, a TUI IRC client written in Rust. Thanks to @ratsclub for helping me write my first Nix module!

Checklist

  • Change is backwards compatible.

  • Code formatted with ./format.

  • Code tested through nix-shell --pure tests -A run.all.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

  • If this PR adds a new module

    • Added myself as module maintainer. See example.

    • Added myself and the module files to .github/CODEOWNERS.

@kmaasrud kmaasrud requested a review from rycee as a code owner February 19, 2022 21:21
Copy link
Copy Markdown
Member

@berbiche berbiche left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

The module follows all our standards 👍
The only nit is that the line added to the CODEOWNERS file is not at the right place.

Comment thread .github/CODEOWNERS Outdated
@kmaasrud
Copy link
Copy Markdown
Contributor Author

I also added the darwin configuration path fix from #2736. tiny seems to respect standard directories, making XDG useless on Darwin.

Comment thread modules/programs/tiny.nix
Comment thread modules/programs/tiny.nix Outdated
@sumnerevans
Copy link
Copy Markdown
Contributor

Looks like you need to add yourself as a maintainer. Note that I just rebased your branch and force-pushed due to unrelated build failures that just got fixed.

@kmaasrud
Copy link
Copy Markdown
Contributor Author

Thanks for that @sumnerevans! Added myself to maintainers.nix now

Comment thread modules/programs/tiny.nix Outdated
Comment thread modules/programs/tiny.nix Outdated
@kmaasrud
Copy link
Copy Markdown
Contributor Author

Sorry about all the fuzz; I'm a new contributor, as you understand 😅

Copy link
Copy Markdown
Contributor

@sumnerevans sumnerevans left a comment

Choose a reason for hiding this comment

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

LGTM

@sumnerevans sumnerevans requested a review from berbiche February 21, 2022 22:00
@berbiche berbiche merged commit 1d90b60 into nix-community:master Feb 26, 2022
teto pushed a commit to teto/home-manager that referenced this pull request Mar 2, 2022
Added module for tiny, a TUI IRC client written in Rust. Thanks to @ratsclub for helping me write my first Nix module!
teto pushed a commit to teto/home-manager that referenced this pull request Aug 22, 2022
Added module for tiny, a TUI IRC client written in Rust. Thanks to @ratsclub for helping me write my first Nix module!
@teto teto mentioned this pull request Aug 22, 2022
7 tasks
spacekookie pushed a commit to spacekookie/home-manager that referenced this pull request Feb 10, 2023
Added module for tiny, a TUI IRC client written in Rust. Thanks to @ratsclub for helping me write my first Nix module!
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.

5 participants