Skip to content

Comments

Add loading of a single level of includes from configuration files#1588

Open
ntninja wants to merge 3 commits intobinwiederhier:mainfrom
ntninja:main
Open

Add loading of a single level of includes from configuration files#1588
ntninja wants to merge 3 commits intobinwiederhier:mainfrom
ntninja:main

Conversation

@ntninja
Copy link

@ntninja ntninja commented Feb 2, 2026

So far I was not able to compile it locally.

Fixes #1492

@ntninja
Copy link
Author

ntninja commented Feb 2, 2026

Well, suggesting go install github.com/goreleaser/goreleaser@latest is a bad idea given it requires over 4GiB to compile and takes who-knows-how-long to build.

@ntninja
Copy link
Author

ntninja commented Feb 2, 2026

Locally built and tested now.

Go remains an appalling minefield of inconsistencies as always, though…

@binwiederhier
Copy link
Owner

It's not Go. It's the CLI library. It's neat but that yaml stuff is wild and confusing

@ntninja
Copy link
Author

ntninja commented Feb 3, 2026

Well, I didn’t interact with the CLI library at all for this, so I’ll take your word on the joy that may have been. For this code it was things like “oh look, structs are passed by value, slices are semantically passed by value but internally passed by reference, but maps are semantically and internally passed by reference” (I fail to see the logic behind this). Or the whole x := expr vs x, ok := expr thing… Or Go’s focus on concurrency, while making race-conditions the default (unless you can use a channel, another tuckered-on odd-ball feature). I just wished Go had been developed with the consistency of D, rather than the exceptionality-ness of JavaScript. It’d be a great language then, oh well…

What’s your take on the code?

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.

Allow loading WebPush secrets from private file

2 participants