Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor/print config #283

Draft
wants to merge 68 commits into
base: main
Choose a base branch
from
Draft

Refactor/print config #283

wants to merge 68 commits into from

Conversation

BinToss
Copy link
Member

@BinToss BinToss commented Mar 6, 2022

Instead of printing a mostly-constant, multi-line string, why not programmatically format a table with the members of Configuration, their description, and their value in the current instance?
Closes #261

BinToss added 30 commits January 9, 2022 00:26
The .NET runtime depependency sni.dll will now be embedded in our app.
Why is it a default dependency of win-x64? and win-x86?
Why is it copied to the Publish folder if it's a prerequisite of the .NET runtime?
I hope this works the way I think it does.
I'm worried the attribute will be set without the file being compressed.
Some faimilar features are not available in .NET Framework.
NOT SUPPORTED IN NET4:
- string.Split(char[] separator, StringSplitOptions options)
- string.Split(string separator)
- System.Net.Http.HttpResponseMessage.Content.ReadAsStream()
Use Microsoft.Win32 or WPF equivalents instead.
style: set charset to UTF-8 BOM
The Windows 7 SingleFile host will be introduced in .NET 6.0.300.
- if both PublishSingleFile and RuntimeIdentifier are unset, disable PublishSingleFile
- if RuntimeIdentifier is unset and PublishSingleFile is enabled, set RuntimeIdentifier to 'win-x86'
- if SelfContained is unset, but RuntimeIdentifier *is*, enable SelfContained
- if PublishSingleFile is enabled..
...enable SelfContained
...enable IncludeNativeLibrariesForSelfExtract
...enable EnableCompressionInSingleFile
This allows halo and haloce cases to
 set LastResult members without one
 case falling through to the other.
...which makes use of recent Result additions.
InferResult() is an informative alternative to Infer.
Infer() is kept for compatibility.
InferResult() is an informative alternative to Infer()
@BinToss
Copy link
Member Author

BinToss commented Mar 26, 2024

I experimented with this again while refactoring feat/AvaloniaUI. I keep running into the same issue: Reflection usage.
I'd prefer avoiding it.
maybe const strings like const string CatConfigurationTweaks = nameof(Category.Tweaks)?

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.

Add descriptions to all config settings
1 participant