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

refactor: Migrate to use graphql-language-service package #1202

Merged
merged 3 commits into from Feb 29, 2024

Conversation

Quramy
Copy link
Owner

@Quramy Quramy commented Feb 29, 2024

What

Uninstall legacy lang service pkgs and migrate to https://www.npmjs.com/package/graphql-language-service

It closes #984

Copy link

codecov bot commented Feb 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.55%. Comparing base (9e65dcf) to head (6a471e0).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1202      +/-   ##
==========================================
- Coverage   92.56%   92.55%   -0.01%     
==========================================
  Files          67       67              
  Lines        4639     4637       -2     
  Branches      570      570              
==========================================
- Hits         4294     4292       -2     
  Misses        345      345              
Files Coverage Δ
src/analyzer/validator.ts 94.11% <100.00%> (ø)
...uage-service-adapter/get-completion-at-position.ts 96.22% <100.00%> (-0.07%) ⬇️
...uage-service-adapter/get-quick-info-at-position.ts 100.00% <100.00%> (ø)
...guage-service-adapter/get-semantic-diagonistics.ts 90.00% <100.00%> (-0.10%) ⬇️
...raphql-language-service-adapter/simple-position.ts 88.00% <100.00%> (ø)

@Quramy Quramy merged commit 6ea0662 into main Feb 29, 2024
3 checks passed
@Quramy Quramy deleted the migrate_lsp_engine branch February 29, 2024 12:56
ckomop0x pushed a commit to ckomop0x/ckomop0x.me.site that referenced this pull request Mar 7, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [ts-graphql-plugin](https://togithub.com/Quramy/ts-graphql-plugin) |
[`3.0.1` ->
`3.0.2`](https://renovatebot.com/diffs/npm/ts-graphql-plugin/3.0.1/3.0.2)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/ts-graphql-plugin/3.0.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/ts-graphql-plugin/3.0.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/ts-graphql-plugin/3.0.1/3.0.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/ts-graphql-plugin/3.0.1/3.0.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>Quramy/ts-graphql-plugin (ts-graphql-plugin)</summary>

###
[`v3.0.2`](https://togithub.com/Quramy/ts-graphql-plugin/releases/tag/v3.0.2)

[Compare
Source](https://togithub.com/Quramy/ts-graphql-plugin/compare/v3.0.1...v3.0.2)

#### Misc changes

- Fix issue template by
[@&#8203;fregante](https://togithub.com/fregante) in
[Quramy/ts-graphql-plugin#1048
- Refactor HTTP test by [@&#8203;Quramy](https://togithub.com/Quramy) in
[Quramy/ts-graphql-plugin#1195
- refactor: Migrate to use graphql-language-service package by
[@&#8203;Quramy](https://togithub.com/Quramy) in
[Quramy/ts-graphql-plugin#1202

#### New Contributors

- [@&#8203;fregante](https://togithub.com/fregante) made their first
contribution in
[Quramy/ts-graphql-plugin#1048

**Full Changelog**:
Quramy/ts-graphql-plugin@v3.0.1...v3.0.2

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ckomop0x/ckomop0x.me.site.v2).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMjAuMiIsInVwZGF0ZWRJblZlciI6IjM3LjIyMC4yIiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIn0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

should depend on/use graphql-language-service instead of graphql-language-service-interface (deprecated)
1 participant