Skip to content

chore: rename overly technical qml command argument#15596

Merged
JoergAtGithub merged 1 commit into
mixxxdj:mainfrom
acolombier:chore/rename-qml-flag
Nov 24, 2025
Merged

chore: rename overly technical qml command argument#15596
JoergAtGithub merged 1 commit into
mixxxdj:mainfrom
acolombier:chore/rename-qml-flag

Conversation

@acolombier
Copy link
Copy Markdown
Member

This addresses #15437

Comment thread src/util/cmdlineargs.cpp Outdated
"profile, or run with "
"'allow-dangerous-data-corruption-risk' to use "
"with the current one. We highly recommend "
"backing up you data if you do so.")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
"backing up you data if you do so.")
"backing up your data if you do so.")

@ronso0
Copy link
Copy Markdown
Member

ronso0 commented Nov 11, 2025

what about next-ui? or next-gen?
Just next sounds like it'd skip someting ; )

@Holzhaus
Copy link
Copy Markdown
Member

what about next-ui? or next-gen?
Just next sounds like it'd skip someting ; )

I have a déjà-vu 😅

See the discussion here: #15513 (comment)

@acolombier
Copy link
Copy Markdown
Member Author

Sorry @ronso0 and @Holzhaus, I should have copied the thread here too! Are you happy with --new-user-interface @ronso0 ?

@ronso0
Copy link
Copy Markdown
Member

ronso0 commented Nov 18, 2025

Are you happy with --new-user-interface @ronso0 ?

Sure, makes sense.
--new-ui for less typing / probability of typos?

@acolombier acolombier force-pushed the chore/rename-qml-flag branch from 89e2a86 to 0eabd0f Compare November 21, 2025 15:48
@acolombier
Copy link
Copy Markdown
Member Author

Renamed the flag now. I took the freedom to amend directly so this can be merged directly

@acolombier acolombier added this to the 3.0-beta milestone Nov 21, 2025
@acolombier
Copy link
Copy Markdown
Member Author

@ronso0 happy to merge?

@ronso0
Copy link
Copy Markdown
Member

ronso0 commented Nov 22, 2025

Yep, LGTM.
Thank you!

@JoergAtGithub
Copy link
Copy Markdown
Member

@acolombier Why you added "needs testing" and "needs review" to this PR? What needs to be done here?

@acolombier
Copy link
Copy Markdown
Member Author

I batch updated all QML PR ready to be reviewed and merged, I think this one is ready to just being merged!

@JoergAtGithub JoergAtGithub merged commit a427b81 into mixxxdj:main Nov 24, 2025
14 checks passed
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.

4 participants