Skip to content

obsidian-headless: init at 0.0.9 #508487

Open
halfwhey wants to merge 7 commits intoNixOS:masterfrom
halfwhey:add-obsidian-headless
Open

obsidian-headless: init at 0.0.9 #508487
halfwhey wants to merge 7 commits intoNixOS:masterfrom
halfwhey:add-obsidian-headless

Conversation

@halfwhey
Copy link
Copy Markdown

Adds obsidian-headless, a headless CLI client for Obsidian Sync and Obsidian Publish.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
    • [NixOS tests] in [nixos/tests].
    • [Package tests] at passthru.tests.
    • Tests in [lib/tests] or [pkgs/test] for functions and "core" functionality.
  • Ran nixpkgs-review on this PR. See [nixpkgs-review usage].
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Fits [CONTRIBUTING.md], [pkgs/README.md], [maintainers/README.md] and other READMEs.

@nixpkgs-ci nixpkgs-ci Bot added 8.has: package (new) This PR adds a new package 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package 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! 9.needs: reviewer This PR currently has no reviewers requested and needs attention. 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` labels Apr 10, 2026
@nixos-discourse
Copy link
Copy Markdown

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/6724

Copy link
Copy Markdown
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

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

Comment thread pkgs/by-name/ob/obsidian-headless/package.nix Outdated
Comment thread pkgs/by-name/ob/obsidian-headless/package.nix Outdated
Comment thread pkgs/by-name/ob/obsidian-headless/package.nix
Comment thread pkgs/by-name/ob/obsidian-headless/package.nix Outdated
Comment thread pkgs/by-name/ob/obsidian-headless/package.nix Outdated
@nixpkgs-ci nixpkgs-ci Bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Apr 12, 2026
@halfwhey halfwhey force-pushed the add-obsidian-headless branch from f2d2015 to 842d795 Compare April 12, 2026 22:37
@halfwhey
Copy link
Copy Markdown
Author

halfwhey commented Apr 12, 2026

All suggestions are applied, binary confirmed to be working.

@SuperSandro2000

Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
@halfwhey halfwhey force-pushed the add-obsidian-headless branch 2 times, most recently from 298d94f to d230d6e Compare April 12, 2026 22:46
@halfwhey
Copy link
Copy Markdown
Author

Addressed issues caught by the workflow

https://github.com/NixOS/nixpkgs/actions/runs/24318091034/job/70999281287?pr=508487

@halfwhey halfwhey force-pushed the add-obsidian-headless branch from d230d6e to 780cb58 Compare April 12, 2026 22:49
@halfwhey
Copy link
Copy Markdown
Author

Let me know if you need me to do anything else @SuperSandro2000

@nixos-discourse
Copy link
Copy Markdown

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/6803

Comment thread pkgs/by-name/ob/obsidian-headless/package.nix Outdated
Copy link
Copy Markdown
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

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

otherwise lgtm

@halfwhey halfwhey requested a review from SuperSandro2000 May 2, 2026 12:42
@halfwhey
Copy link
Copy Markdown
Author

halfwhey commented May 2, 2026

Requested changes made

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package 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.first-time contribution This PR is the author's first one; please be gentle!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants