Skip to content

Overhaul ArgumentParser argument parsing#731

Merged
Spartan322 merged 1 commit into
masterfrom
overhaul/argument-parser
Jul 19, 2026
Merged

Overhaul ArgumentParser argument parsing#731
Spartan322 merged 1 commit into
masterfrom
overhaul/argument-parser

Conversation

@Spartan322

@Spartan322 Spartan322 commented Jul 18, 2026

Copy link
Copy Markdown
Member

Add ignore for conflicting Godot Engine options (like -s and -d)
Extract argument boolean list to function
Extract argument with option =value parsing to function
Add explicit whitespaced boolean option disable
Streamline parse_arguments
Add conflict_type to ArgumentOption for specifying Godot Engine option conflicts

@Spartan322
Spartan322 requested a review from a team as a code owner July 18, 2026 01:17
@Spartan322 Spartan322 added enhancement Issue that describes an improvement to the project topic:core Issue that applies to core functions, fundamental to the rest of the module labels Jul 18, 2026
Add ignore for conflicting Godot Engine options (like -s and -d)
Extract argument boolean list to function
Extract argument with option `=value` parsing to function
Add explicit whitespaced boolean option disable
Streamline parse_arguments
Add conflict_type to ArgumentOption for specifying Godot Engine option conflicts
@Spartan322
Spartan322 force-pushed the overhaul/argument-parser branch from ba24333 to cde457a Compare July 19, 2026 10:15
@Spartan322
Spartan322 added this pull request to the merge queue Jul 19, 2026
Merged via the queue into master with commit 7f6ee1a Jul 19, 2026
11 checks passed
@Spartan322
Spartan322 deleted the overhaul/argument-parser branch July 19, 2026 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Issue that describes an improvement to the project topic:core Issue that applies to core functions, fundamental to the rest of the module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants