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

Adopt ansible-language-server as a workspace #1135

Merged
merged 606 commits into from Mar 15, 2024
Merged

Adopt ansible-language-server as a workspace #1135

merged 606 commits into from Mar 15, 2024

Conversation

ssbarnea
Copy link
Member

@ssbarnea ssbarnea commented Mar 15, 2024 •
edited

priyamsahoo and others added 30 commits March 29, 2022 22:13
@priyamsahoo
Merge pull request #278 from ansible/dependabot/pip/click-8.1.0
c029883
@ganeshrn @pre-commit-ci
Fix EE auto-completion not working issue (#279)
85ebefc 
* Fix EE auto-completion not working issue

*  Currently fetch plugin from EE requires
   ansibleConfig service to be invoked to identify
   the path within EE that contains ansible plugins

*  Also docLibrary service invokes ansibleConfig
   service thus duplicating the call

*  The PR refactors the code to invoke fetching
   EE plugin docs from within docsLibrary service
   and thus avoiding multiple async invocation of
   ansibleConfig service

* add changelog

* chore: auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@priyamsahoo @pre-commit-ci @ganeshrn
Resolve completion of module options and sub options based on their t…
df45a1a 
…ype (#276)

* provide '-' when option or suboption is a list

* replace double quotes with single quotes after latest merge

* add logic for appropriate resolution of module options and suboptions based on their types

* add double tabs on resolution of dict to avoid wrong autocompletion if the first element of the list is dict

* add basic structure for doResolveCompletion test

* update quotations

* remove ':' after option name when not found at end of line while
resolution

* add test fixture of doCompletionResolve() tests

* update tests for doCompletionResolve()

* add type for parameter in resolveSuffix function

* chore: auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* fix linting

* add change note

* add ability to give default insertText when 'type' is not present for
module options

* add utility function to add data if an option or module name is the
first element of list or not

* call resolveSuffix for module name by sending type as 'dict' for them

* update test script and fixture with latest changes in doCompletionResolve()

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ganesh Nalawade <ganesh634@gmail.com>
@ssbarnea
Add ansible-lint to test requirements lockfile (#272)
aa5151f
@dependabot
Bump click from 8.1.0 to 8.1.1 (#281)
7717873 
Bumps [click](https://github.com/pallets/click) from 8.1.0 to 8.1.1.
- [Release notes](https://github.com/pallets/click/releases)
- [Changelog](https://github.com/pallets/click/blob/main/CHANGES.rst)
- [Commits](pallets/click@8.1.0...8.1.1)

---
updated-dependencies:
- dependency-name: click
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump ansible-core from 2.12.3 to 2.12.4 (#282)
64c43ec 
Bumps [ansible-core](https://github.com/ansible/ansible) from 2.12.3 to 2.12.4.
- [Release notes](https://github.com/ansible/ansible/releases)
- [Commits](ansible/ansible@v2.12.3...v2.12.4)

---
updated-dependencies:
- dependency-name: ansible-core
  dependency-type: indirect
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@yaegassy
fix: add insertTextFormat to completionItem
3c049b7
@yaegassy
chore: add change note (285)
b99b977
@pre-commit-ci
chore: auto fixes from pre-commit.com hooks
29e51b0 
for more information, see https://pre-commit.ci
@yaegassy
fix: add change note (285)
3edec67
@pre-commit-ci
chore: auto fixes from pre-commit.com hooks
445b558 
for more information, see https://pre-commit.ci
@priyamsahoo
Add neovim to dictionary (#286)
33c358b
@priyamsahoo
Merge branch 'main' into fix/add-insert-text-format
d5b3b28
@priyamsahoo
Merge pull request #285 from yaegassy/fix/add-insert-text-format
0959102
@github-actions @pre-commit-ci
Bump version to v0.5.4 (#283)
3e45bd2 
* Generate a changelog entry for v0.5.4

* Release 0.5.4 from GitHub Actions CI/CD

* chore: auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@tomaciazek @pre-commit-ci @ganeshrn
Autocomplete option and suboption choices (#288)
bab8ec7 
* Autocomplete option and suboption choices

* chore: auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix tests & add more

* Fix tests

* Add changelog fragment

* chore: auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ganesh Nalawade <ganesh634@gmail.com>
@dependabot
Bump pre-commit from 2.17.0 to 2.18.1 (#291)
43aaa61 
Bumps [pre-commit](https://github.com/pre-commit/pre-commit) from 2.17.0 to 2.18.1.
- [Release notes](https://github.com/pre-commit/pre-commit/releases)
- [Changelog](https://github.com/pre-commit/pre-commit/blob/main/CHANGELOG.md)
- [Commits](pre-commit/pre-commit@v2.17.0...v2.18.1)

---
updated-dependencies:
- dependency-name: pre-commit
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@pre-commit-ci @ssbarnea
chore: pre-commit autoupdate (#292)
937df83 
updates:
- [github.com/streetsidesoftware/cspell-cli: v5.8.4 → v5.9.0](streetsidesoftware/cspell-cli@v5.8.4...v5.9.0)
- [github.com/pre-commit/mirrors-prettier: v2.5.1 → v2.6.2](pre-commit/mirrors-prettier@v2.5.1...v2.6.2)
- [github.com/pre-commit/mirrors-eslint: v8.10.0 → v8.12.0](pre-commit/mirrors-eslint@v8.10.0...v8.12.0)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Sorin Sbarnea <ssbarnea@redhat.com>
@priyamsahoo @pre-commit-ci
Utility to update settings in readme automatically (#294)
a645d39 
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@ssbarnea
Ensure we keep generated settings docs in sync (#296)
d389c42
@dependabot
Bump click from 8.1.1 to 8.1.2 (#287)
407b706 
Bumps [click](https://github.com/pallets/click) from 8.1.1 to 8.1.2.
- [Release notes](https://github.com/pallets/click/releases)
- [Changelog](https://github.com/pallets/click/blob/main/CHANGES.rst)
- [Commits](pallets/click@8.1.1...8.1.2)

---
updated-dependencies:
- dependency-name: click
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@priyamsahoo @pre-commit-ci
Replace globby with glob (#295)
e48724a 
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@ganeshrn @pre-commit-ci
Move glob utils to different utils file (#299)
1e18764 
* Move glob utils to different utils file

*  Move glob utils function into a path utils file
*  Remove commented out code

* chore: auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* add changelog file

* fix ci failure and rename test file

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@ganeshrn @ssbarnea
Add vscode extension e2e test in CI (#289)
50a0d41 
* Add vscode extension e2e test in CI

*  Add CI job to run test-e2e suite from
   vscode-ansible extension to ensure
   changes in PR does not break vscode-ansible
   extension

* add new line

* Update vscode.yml

* update vscode workflow

* install ansible-language-server dependecies

* install test dependecies

* update test name

Co-authored-by: Sorin Sbarnea <sorin.sbarnea@gmail.com>
Co-authored-by: Sorin Sbarnea <ssbarnea@redhat.com>
@ssbarnea
Refactor user of python dependencies (#298)
775f7bd 
See: https://github.com/ansible/devtools/wiki/tox#dependency-pinning
@ganeshrn
Add utility to sync als version for devel CI job (#301)
dfbebfb 
* Add utility to sync als version for devel CI job

*  Add python utility to copy ansible-language-server
   version to the vscode-ansible pacakage.json file
   for testing with main branch

* fix ci failure

* remove unwanted line
@github-actions @pre-commit-ci @ssbarnea
Bump version to v0.6.0 (#300)
5fa5a1e 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Sorin Sbarnea <ssbarnea@redhat.com>
@ssbarnea
Remove release cleanup pipeline (#302)
23dd56f
@ssbarnea
Fix execution of tox -e deps (#303)
78a2bf4
@priyamsahoo
move handlebars to dev dependencies (#305)
277aa8c
@ssbarnea
Update yarn and vscode-extension-tester
d59ad63
@ssbarnea
Remove devel pipeline
0c6b28f 
Devel pipeline is no longer needed because we always build
ansible-language-server from inside the repo.
@ssbarnea ssbarnea marked this pull request as ready for review March 15, 2024 17:12
@ssbarnea ssbarnea requested a review from goneri March 15, 2024 17:14
@ssbarnea ssbarnea merged commit aeecbf9 into main Mar 15, 2024
8 checks passed
@ssbarnea ssbarnea deleted the join branch March 15, 2024 17:34
@TamiTakamiya TamiTakamiya mentioned this pull request Mar 18, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Qalthos Qalthos Awaiting requested review from Qalthos Qalthos 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

@ganeshrn ganeshrn Awaiting requested review from ganeshrn

@TamiTakamiya TamiTakamiya Awaiting requested review from TamiTakamiya

@priyamsahoo priyamsahoo Awaiting requested review from priyamsahoo

@goneri goneri Awaiting requested review from goneri

Assignees

@ssbarnea ssbarnea

Labels
skip-changelog
Projects
🧰 devtools project board
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet