Skip to content
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

Global settings #41

Merged
merged 3 commits into from
Jun 18, 2022
Merged

Global settings #41

merged 3 commits into from
Jun 18, 2022

Conversation

cljoly
Copy link
Owner

@cljoly cljoly commented Jun 4, 2022

  • feat: Add global settings for the commands
  • doc: document the new global configuration
  • style: run stylua

See the additional documentation in the Readme for usage instruction of this new feature.

@cljoly cljoly added documentation Improvements or additions to documentation enhancement New feature or request labels Jun 4, 2022
@@ -106,7 +106,50 @@ use {

## Usage

### list
### Global Configuration
Copy link
Owner Author

@cljoly cljoly Jun 4, 2022

Choose a reason for hiding this comment

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

@Chetan496 feel free to point out anything unclear in that section or anything that does not work for you. (this is re: #38)

Choose a reason for hiding this comment

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

@Chetan496 feel free to point out anything unclear in that section or anything that does not work for you. (this is re: #38)

Thanks for this, this is very helpful.
I will try this feature soon.

@cljoly cljoly force-pushed the global-settings branch from a47bcc0 to daaac40 Compare June 4, 2022 13:55
@Chetan496
Copy link

Thanks a lot for this feature, will try it soon

@cljoly
Copy link
Owner Author

cljoly commented Jun 4, 2022

Great! I’m happy to refine this feature based on your input, feel free to share your thoughts on this.

@cljoly cljoly mentioned this pull request Jun 5, 2022
@Chetan496
Copy link

It would be great to have this merged into master/main branch

@cljoly
Copy link
Owner Author

cljoly commented Jun 18, 2022

Sorry, I forgot that PR

@cljoly cljoly merged commit 867f87a into dev Jun 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants