Skip to content

fix(nix): add pyright to devshell - #1415

Merged
xarvex merged 2 commits into
mainfrom
fix-nix-add-pyright
Jul 2, 2026
Merged

fix(nix): add pyright to devshell#1415
xarvex merged 2 commits into
mainfrom
fix-nix-add-pyright

Conversation

@Computerdores

Copy link
Copy Markdown
Collaborator

Summary

If you don't have pyright installed on the system, the commit hook in the dev shell will fail.
This PR adds pyright to the devshell to fix that issue.

Tasks Completed

  • Platforms Tested:
    • Windows x86
    • Windows ARM
    • macOS x86
    • macOS ARM
    • Linux x86
    • Linux ARM
  • Tested For:
    • Basic functionality
    • PyInstaller executable

@Computerdores
Computerdores requested a review from xarvex July 2, 2026 18:34
@Computerdores Computerdores moved this to 🏓 Ready for Review in TagStudio Development Jul 2, 2026
@xarvex

xarvex commented Jul 2, 2026

Copy link
Copy Markdown
Member

Just did a quick switch-a-roo to put this in nativeBuildInputs (see this answer for why). Thanks!

@xarvex
xarvex merged commit 00b6f91 into main Jul 2, 2026
12 checks passed
@github-project-automation github-project-automation Bot moved this from 🏓 Ready for Review to ✅ Done in TagStudio Development Jul 2, 2026
@Computerdores

Copy link
Copy Markdown
Collaborator Author

Just did a quick switch-a-roo to put this in nativeBuildInputs (see this answer for why). Thanks!

Could you elaborate on that? Because I always use buildInputs for all my devshells and have never had problems with that and the post seems to suggest that it just doesn't make a difference

@CyanVoxel CyanVoxel added Type: Installation Installing, building, and/or launching the program System: Linux For Linux/BSD distributions labels Jul 4, 2026
@Computerdores
Computerdores deleted the fix-nix-add-pyright branch July 5, 2026 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

System: Linux For Linux/BSD distributions Type: Installation Installing, building, and/or launching the program

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants