Skip to content

Add optional=true to niri configs#2776

Merged
Purian23 merged 4 commits into
masterfrom
nix-optional-includes
Jul 8, 2026
Merged

Add optional=true to niri configs#2776
Purian23 merged 4 commits into
masterfrom
nix-optional-includes

Conversation

@LuckShiba

@LuckShiba LuckShiba commented Jul 8, 2026

Copy link
Copy Markdown
Member

Description

Closes #1534. Adds optional=true to the includes in DMS niri-flake module and embedded/niri.kdl config. Now that niri 26.04 is mainstreamed, we can add this to avoid niri config parsing failing for still not generated files, such as in #1534.

Also updated flake.lock as a routinary action.

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that changes existing behavior)
  • Refactor / internal cleanup
  • Documentation
  • Other

Related issues

Screenshots / video

Checklist

  • My code follows the conventions in CONTRIBUTING.md
  • I have tested my changes locally
  • New user-facing strings are wrapped in I18n.tr() with translator context, reusing existing terms where possible
  • Go changes: ran make fmt, added/updated tests, make test passes, and go mod tidy is clean
  • QML changes: ran make lint-qml with no new warnings
  • I have opened a corresponding pull request in dlx-docs to document any new behaviors: https://github.com/AvengeMedia/DankLinux-Docs

@LuckShiba LuckShiba changed the title Add optional=true to nix niri-flake module Add optional=true to niri configs Jul 8, 2026
@Purian23 Purian23 merged commit 56712f4 into master Jul 8, 2026
5 checks passed
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.

niri.includes.filesToInclude default includes outputs but DMS doesn't generate outputs.kdl for niri

2 participants