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

Update inline suggestion trigger condition (#890) #892

Merged
merged 1 commit into from May 16, 2023

Conversation

ganeshrn
Copy link
Member

  • Currently inline suggestion is triggered after user hits new line key at end of the task name description. So on every space/tab a lightspeed request is triggered which is not expected.
  • With this fix the request will be triggered only when the number of spaces before cursor position on new line is equal to the number of spaces before - in the previous line (task name description).

*  Currently inline suggestion is triggered after user
   hits new line key at end of the task name description.
   So on every space/tab a lightspeed request is triggered which
   is not expected.
*  With this fix the request will be triggered only when the number of
   spaces before cursor position on new line is equal to the number of
   spaces before `-` in the previous line (task name description).
@ganeshrn ganeshrn requested a review from a team as a code owner May 16, 2023 13:21
@ganeshrn ganeshrn added bug Something isn't working AAP Ansible automation platform labels May 16, 2023
Copy link
Contributor

@priyamsahoo priyamsahoo left a comment

Choose a reason for hiding this comment

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

looks good to me.

@ganeshrn ganeshrn merged commit 94dd63b into main May 16, 2023
8 of 9 checks passed
@ganeshrn ganeshrn deleted the lightspeed_suggestion branch May 16, 2023 18:15
github-actions bot pushed a commit that referenced this pull request May 18, 2023
## v2.1

### Minor Changes
- Easy discovery of Python interpreter and Ansible executables (#871) @priyamsahoo

### Bugfixes

- Add information message for inline suggestion trigger command (#896) @ganeshrn
- Update status-bar items (#891) @priyamsahoo
- Update inline suggestion trigger condition (#892) @ganeshrn
- Add ability to select and set cursor on vars (#883) @priyamsahoo
- Add logic for conditional Authentication Provider registration (#877) @ganeshrn
- Fix regex to match trigger for lightspeed suggestion (#875) @ganeshrn
- Update dependencies (#872) @ssbarnea
@ansibot ansibot mentioned this pull request May 18, 2023
ansibot added a commit that referenced this pull request May 18, 2023
## v2.1

### Minor Changes
- Easy discovery of Python interpreter and Ansible executables (#871) @priyamsahoo

### Bugfixes

- Add information message for inline suggestion trigger command (#896) @ganeshrn
- Update status-bar items (#891) @priyamsahoo
- Update inline suggestion trigger condition (#892) @ganeshrn
- Add ability to select and set cursor on vars (#883) @priyamsahoo
- Add logic for conditional Authentication Provider registration (#877) @ganeshrn
- Fix regex to match trigger for lightspeed suggestion (#875) @ganeshrn
- Update dependencies (#872) @ssbarnea

Co-authored-by: Ansible DevTools <ansible-devtools@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AAP Ansible automation platform bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants