Skip to content

option source permalinks#1581

Open
mightyiam wants to merge 3 commits intohercules-ci:mainfrom
mightyiam:option-permalinks
Open

option source permalinks#1581
mightyiam wants to merge 3 commits intohercules-ci:mainfrom
mightyiam:option-permalinks

Conversation

@mightyiam
Copy link
Copy Markdown
Contributor

  • move a let in
  • option source permalinks

@mightyiam mightyiam force-pushed the option-permalinks branch 2 times, most recently from b46d237 to 3cc74b8 Compare July 2, 2025 18:12
Comment thread flake.nix
Comment thread core-modules.nix
Comment thread flake.nix
Comment thread flake.nix
Comment thread flake.nix
Comment thread flake.nix
Comment thread core-modules.nix
Comment thread render/render-module.nix
Comment thread render/render-module.nix Outdated
@mightyiam mightyiam marked this pull request as ready for review July 2, 2025 18:16
@mightyiam
Copy link
Copy Markdown
Contributor Author

Oh and if you review the two commits separately it's a little bit easier.

@mightyiam mightyiam force-pushed the option-permalinks branch from 3cc74b8 to ffb4be1 Compare July 21, 2025 15:22
Copy link
Copy Markdown
Member

@roberth roberth left a comment

Choose a reason for hiding this comment

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

TODO later: find a way to document importable options
1 suggestion
The rest LGTM!

Comment thread render/render-module.nix Outdated
Comment thread render/render-module.nix Outdated
@mightyiam mightyiam force-pushed the option-permalinks branch from ffb4be1 to f754906 Compare July 21, 2025 15:56
@mightyiam mightyiam requested a review from roberth July 23, 2025 04:54
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