-
Notifications
You must be signed in to change notification settings - Fork 217
add max-suggestion config #387
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
base: main
Are you sure you want to change the base?
add max-suggestion config #387
Conversation
@microsoft-github-policy-service agree |
|
Hi @cpendery ! can you please review this and merge or let me know if any changes are required ? |
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.
Can you also add a screenshot of the change working in the PR description & add a readme section under configuration explaining the configuration setting, similar to the useAliases / useNerdFonts
|
it looks like last few changes are breaking npm run dev before syncing the fork it worked correctly and now it asks for version.txt even in dev mode , shall I build a package and then test this or we are going to fix this @cpendery ? |
for now I will build a package on local and attach a ss |
|
Hi @cpendery took some time but done!! |
closes #373

We can now see more or less than 5 suggestions in the suggestion box
See below