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

doc(action_state): improve description of action_state.get_current_line #2895

Merged
merged 2 commits into from
Jan 27, 2024

Conversation

Kobu
Copy link
Contributor

@Kobu Kobu commented Jan 26, 2024

Description

I have been trying for quite some time to find this method in docs, but it was nearly improssible. The description should contain either "input" or "prompt", because those are the keywords that most people would search for when trying to find this method.

Type of change

Please delete options that are not relevant.

  • This change requires a documentation update

How Has This Been Tested?

No test, just docs update

Configuration:
NVIM v0.9.5
Build type: Release
LuaJIT 2.1.1702233742

Checklist:

  • My code follows the style guidelines of this project (stylua)
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation (lua annotations)

@jamestrew
Copy link
Contributor

Thanks, this is a welcome addition.
But our docs are auto-gen'ed from the docstrings in our lua code. You will have to make the change there.
Or I can do it later as well if need be.

@Kobu
Copy link
Contributor Author

Kobu commented Jan 27, 2024

Sorry, I missed that.

Changed the docstring in lua code instead and dropped the previous commit. Should be all good now

@Conni2461
Copy link
Member

thanks :)

@Conni2461 Conni2461 merged commit f1fd99e into nvim-telescope:master Jan 27, 2024
6 checks passed
Conni2461 pushed a commit that referenced this pull request Mar 11, 2024
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.

3 participants