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.

Sign up for GitHub

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

Jump to bottom

Easy discovery of Python interpreter and Ansible executables #871

Merged
merged 15 commits into from May 5, 2023
Merged

Easy discovery of Python interpreter and Ansible executables #871

merged 15 commits into from May 5, 2023

Conversation

priyamsahoo
Copy link
Contributor

The PR:

  1. Adds the ability to communicate with the Python extension and sets the interpreter based on the recommended one by the Python extension.
  2. Adds feature to select interpreter on the go by providing options for available interpreters (similar to Python's select interpreter feature).
  3. fixes: Improve ansible* executable discovery #775

@priyamsahoo priyamsahoo added the feature New feature or request label May 3, 2023
@priyamsahoo priyamsahoo requested a review from a team as a code owner May 3, 2023 10:43
@priyamsahoo priyamsahoo changed the title initial commit for easy python discovery work Easy discovery of Python interpreter and Ansible executables May 3, 2023
@priyamsahoo priyamsahoo merged commit a5acf83 into ansible:main May 5, 2023
6 of 7 checks passed
github-actions bot pushed a commit that referenced this pull request May 18, 2023
@ansibuddy
Release v2.1
15902a5 
## 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
Merged
ansibot added a commit that referenced this pull request May 18, 2023
@ansibot @ansibuddy
Release v2.1 (#899)
f6f731e 
## 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>
@pluto00987 pluto00987 mentioned this pull request May 22, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ganeshrn ganeshrn ganeshrn approved these changes

@ssbarnea ssbarnea Awaiting requested review from ssbarnea ssbarnea is a code owner automatically assigned from ansible/devtools

@audgirka audgirka Awaiting requested review from audgirka audgirka is a code owner automatically assigned from ansible/devtools

@shatakshiiii shatakshiiii Awaiting requested review from shatakshiiii shatakshiiii is a code owner automatically assigned from ansible/devtools

Assignees
No one assigned
Labels
feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Improve ansible* executable discovery
2 participants
@priyamsahoo @ganeshrn