Skip to content

ios-notif-forward: init at 0.1.1 #432603

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

ios-notif-forward: init at 0.1.1 #432603

wants to merge 2 commits into from

Conversation

howtonotwin
Copy link
Contributor

ios-notif-forward is a small Rust application that looks for iOS devices over Bluetooth and forwards notifications from them as desktop notifications. I've found it convenient and think it'd be nice to have in Nixpkgs.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.first-time contribution This PR is the author's first one; please be gentle! 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` labels Aug 10, 2025
@howtonotwin howtonotwin force-pushed the init-ios-notif-forward branch 2 times, most recently from 9c1dd28 to e314d63 Compare August 10, 2025 19:45
@nixpkgs-ci nixpkgs-ci bot added the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Aug 10, 2025
acid-bong
acid-bong suggested changes Aug 11, 2025
View reviewed changes
@nixpkgs-ci nixpkgs-ci bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Aug 11, 2025
@acid-bong
Copy link
Contributor

And a general request: keep only two commits (the following are not titles)

  1. new maintainer
  2. new package

All minor edits should be amended/squashed to the respective commits

@howtonotwin howtonotwin force-pushed the init-ios-notif-forward branch 2 times, most recently from fb17236 to e579c3b Compare August 11, 2025 13:30
@howtonotwin howtonotwin requested a review from acid-bong August 11, 2025 13:38
@acid-bong
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 432603

Logs: https://github.com/acid-bong/nixpkgs-review-gha/actions/runs/16882494644

x86_64-linux

✅ 1 package built:
  • ios-notif-forward

aarch64-linux

✅ 1 package built:
  • ios-notif-forward

acid-bong
acid-bong approved these changes Aug 11, 2025
View reviewed changes
Copy link
Contributor

@acid-bong acid-bong left a comment

Choose a reason for hiding this comment

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

Approved automatically following the successful run of nixpkgs-review.

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. and removed 12.first-time contribution This PR is the author's first one; please be gentle! labels Aug 11, 2025
@howtonotwin howtonotwin force-pushed the init-ios-notif-forward branch from e579c3b to 8d50172 Compare August 19, 2025 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@acid-bong acid-bong acid-bong approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@howtonotwin @acid-bong