Skip to content

languages/docker: add missing mkIf#1592

Merged
snoweuph merged 1 commit into
NotAShelf:mainfrom
snoweuph:fix/docker
May 14, 2026
Merged

languages/docker: add missing mkIf#1592
snoweuph merged 1 commit into
NotAShelf:mainfrom
snoweuph:fix/docker

Conversation

@snoweuph
Copy link
Copy Markdown
Collaborator

Minor fix for #1431

Sanity Checking

  • I have updated the changelog as per my changes
  • I have tested, and self-reviewed my code
  • My changes fit guidelines found in hacking nvf
  • Style and consistency
    • I ran Alejandra to format my code (nix fmt)
    • My code conforms to the editorconfig configuration of the project
    • My changes are consistent with the rest of the codebase
  • If new changes are particularly complex:
    • My code includes comments in particularly complex areas
    • I have added a section in the manual
    • (For breaking changes) I have included a migration guide
  • Package(s) built:
    • .#nix (default package)
    • .#maximal
    • .#docs-html (manual, must build)
    • .#docs-linkcheck (optional, please build if adding links)
  • Tested on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

Add a 👍 reaction to pull requests you find important.

@github-actions
Copy link
Copy Markdown

🚀 Live preview deployed from 6b8e926

View it here:

Debug Information

Triggered by: snoweuph

HEAD at: fix/docker

Reruns: 2955

Copy link
Copy Markdown
Collaborator

@Soliprem Soliprem left a comment

Choose a reason for hiding this comment

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

damn

@snoweuph snoweuph merged commit 56ea692 into NotAShelf:main May 14, 2026
14 checks passed
@snoweuph snoweuph deleted the fix/docker branch May 14, 2026 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants