-
Notifications
You must be signed in to change notification settings - Fork 480
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
apps/nshlib: Never disable HELP and ? #2344
Conversation
@xiaoxiang781216 could you please merge? |
but defconfig from stm32f103-minimum/nsh need update too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The commit message and PR description are a little confusing. Instead of "Never disable HELP and ?" I recommend "Enable HELP and ? by default."
@acassis please rebase your patch to pass ci. |
Done |
It is same as 0 and "Oh", I used the name of CONFIG_NSH_DISABLE_HELP as reference. It is not disabling HELP, but internally it could be disabled if others requirements are not met |
Summary
Never disable HELP and ?
Impact
Users will be able to see available commands in NSH
Testing
stm32f103-minimum
Please ignore the CI failing, it needs apache/nuttx#12038 because default board config changed (chichen-egg dilemma).