Skip to content

Conversation

Rishik-Y
Copy link
Member

@Rishik-Y Rishik-Y commented Dec 24, 2024

Package Information

  • Package name: Termius-beta
  • Latest released version: 9.9.0
  • Current version on the unstable channel: none
  • Current version on the stable/release channel: none

Notification for Termius Maintainer:
@Br1ght0ne @Th0rgal
Hi, I have utilized code from Termius and modified it for Termius-beta to work, and as far as testing goes Termius-beta works completely fine.
Since the code is very similar, i have kept both of you as Maintainers for Termius-beta as well.
If you don't want to be maintainer for this code, its completely fine just do notify me 🤗

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.

@Rishik-Y
Copy link
Member Author

Sorry about the error,
Probably due to me not adding myself to Maintainer list,
Which i did just now! 🫠

Copy link
Contributor

@drupol drupol left a comment

Choose a reason for hiding this comment

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

Hello !

Thanks for your contribution!

A couple of tips:

  • This PR must contains 2 commits only:
    • A commit to add you in the maintainer list
    • A commit to add Termius-beta
    • No merge commits
  • You must pass the new nix files through nixfmt-rfc-style

Once this is done, I'll review the content of the nix file.

@Rishik-Y
Copy link
Member Author

Hi @drupol, I am sorry to ask of you this but i just closed this PR and created a new one:
#368256
Although did follow your tips tho..

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