Skip to content

ALL must be uppercase#39

Merged
mdegat01 merged 1 commit intomainfrom
all-to-ALL
Dec 20, 2024
Merged

ALL must be uppercase#39
mdegat01 merged 1 commit intomainfrom
all-to-ALL

Conversation

@mdegat01
Copy link
Copy Markdown
Collaborator

@mdegat01 mdegat01 commented Dec 20, 2024

Proposed Changes

All addons flag needs ALL not all

@mdegat01 mdegat01 added the bugfix Inconsistencies or issues which will cause a problem for users or implementers. label Dec 20, 2024
@mdegat01 mdegat01 merged commit 3d8a26e into main Dec 20, 2024
@mdegat01 mdegat01 deleted the all-to-ALL branch December 20, 2024 13:11
emontnemery
emontnemery reviewed Dec 20, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@emontnemery emontnemery left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot locked and limited conversation to collaborators Dec 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@emontnemery emontnemery emontnemery left review comments

Assignees

No one assigned

Labels

bugfix Inconsistencies or issues which will cause a problem for users or implementers.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mdegat01 @emontnemery