Skip to content

Include app on default module#2

Merged
vic merged 1 commit into
mainfrom
vic/jj-change-onrroprlymuk
Jul 3, 2025
Merged

Include app on default module#2
vic merged 1 commit into
mainfrom
vic/jj-change-onrroprlymuk

Conversation

@vic
Copy link
Copy Markdown
Member

@vic vic commented Jul 3, 2025

No description provided.

Copilot AI review requested due to automatic review settings July 3, 2025 10:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR integrates the new write-files.nix helper into both the dendritic and default modules and updates the README to reflect the changed setup steps.

  • Introduces write-files.nix into modules/dendritic.nix and modules/default.nix imports
  • Revises README: renames step “Create Module” to “Move Inputs” and updates import examples

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
modules/dendritic.nix Replaced files.nix import with write-files.nix
modules/default.nix Added ./dendritic/write-files.nix to imports
README.md Updated step numbering, renamed step 4, and adjusted import examples
Comments suppressed due to low confidence (1)

modules/dendritic.nix:2

  • The import of ./dendritic/files.nix was removed but may still be required by write-files.nix or other components. Please verify that all dependencies formerly provided by files.nix are covered or intentionally deprecated.
  imports = [

Comment thread modules/default.nix Outdated
@vic vic force-pushed the vic/jj-change-onrroprlymuk branch 3 times, most recently from 286857e to d980318 Compare July 3, 2025 10:52
@vic vic force-pushed the vic/jj-change-onrroprlymuk branch from d980318 to 0538c75 Compare July 3, 2025 10:54
@vic vic merged commit 5769e40 into main Jul 3, 2025
2 checks passed
@vic vic deleted the vic/jj-change-onrroprlymuk branch July 3, 2025 10:55
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.

2 participants