Skip to content

Add shmSize to management flag's init#1603

Merged
katiewasnothere merged 1 commit into
apple:mainfrom
katiewasnothere:management_flags_init
May 27, 2026
Merged

Add shmSize to management flag's init#1603
katiewasnothere merged 1 commit into
apple:mainfrom
katiewasnothere:management_flags_init

Conversation

@katiewasnothere
Copy link
Copy Markdown
Contributor

Type of Change

  • Bug fix

Motivation and Context

Ensure all fields are initialized in the management flags' init function. Without this change, if someone calls init() on this set of flags, they will get an error like

Can't read a value from a parsable argument definition.

This error indicates that a property declared with an `@Argument`,
`@Option`, `@Flag`, or `@OptionGroup` property wrapper was neither
initialized to a value nor decoded from command-line arguments.

To get a valid value, either call one of the static parsing methods
(`parse`, `parseAsRoot`, or `main`) or define an initializer that
initializes _every_ property of your parsable type.

Testing

  • Tested locally

Signed-off-by: Kathryn Baldauf <k_baldauf@apple.com>
@github-actions
Copy link
Copy Markdown

Code Coverage

Tier Line Coverage
Unit 33.52%
Integration 19.55%
Combined 52.51%

@katiewasnothere katiewasnothere merged commit 8261a27 into apple:main May 27, 2026
4 checks passed
@katiewasnothere katiewasnothere deleted the management_flags_init branch May 27, 2026 00:51
Copy link
Copy Markdown

@hernandezadonai587 hernandezadonai587 left a comment

Choose a reason for hiding this comment

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

Q

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.

3 participants