merge origin#2
Open
yagui wants to merge 20 commits into
Open
Conversation
- From command line --mark-when-checked - In the config: mark_when_checked - JS: config.mark_when_checked - GUI: None
Add get_wx() in compat.py to gate wx imports behind INTERACTIVE_HTML_BOM_NO_DISPLAY environment variable.
Old algorithm that used zone min thickness in addition to polygon was retired in 7.0
It may cause issues on windows with network paths Fixes #516
Also a bug where default variant exclude from bom was overriding the flag value from selected variant Fixes #544
* Add ini file parsing * Fix ini option types and add missing options - highlight_pin1 is a string choice, not a bool; migrate legacy 0/1 values same as load_from_ini - blacklist_empty_val is a bool in the ini file - board_rotation is stored as int in 5-degree units; convert the CLI fallback value the same way set_from_args does - read offset_back_rotation and mark_when_checked, which are written to the ini file but were not read back - exit with ERROR_FILE_NOT_FOUND instead of 0 when no ini file exists Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * Fix ini/CLI precedence so command line options override ini Explicitly-provided command line options now take precedence over values loaded from the ini file. * Reduce duplicated ini parsing logic in set_from_args_with_ini Replace the repeated per-option ini lookups with a single declarative table of (section, key, attribute, kind, cli_dest) tuples processed by a common loop. Behaviour is unchanged; special cases (highlight_pin1 migration and show_fields/group_fields with --extra-fields) are kept explicit. * Refactor argument parsing to use ini file as argparse defaults --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.