Skip to content

Conversation

@Foo-x
Copy link
Contributor

@Foo-x Foo-x commented Jul 31, 2022

Extracted the logic that creating diagnostics to a method.

@changeset-bot
Copy link

changeset-bot bot commented Jul 31, 2022

⚠️ No Changeset found

Latest commit: 35581eb

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codecov
Copy link

codecov bot commented Jul 31, 2022

Codecov Report

Merging #2609 (35581eb) into main (2d91916) will increase coverage by 3.84%.
The diff coverage is 23.84%.

@@            Coverage Diff             @@
##             main    #2609      +/-   ##
==========================================
+ Coverage   65.70%   69.54%   +3.84%     
==========================================
  Files          85       72      -13     
  Lines        5106     4266     -840     
  Branches     1631     1432     -199     
==========================================
- Hits         3355     2967     -388     
+ Misses       1747     1294     -453     
- Partials        4        5       +1     
Impacted Files Coverage Δ
packages/codemirror-graphql/src/lint.ts 100.00% <ø> (ø)
packages/codemirror-graphql/src/results/mode.ts 47.05% <ø> (ø)
packages/codemirror-graphql/src/utils/hintList.ts 95.65% <ø> (ø)
...ckages/codemirror-graphql/src/utils/mode-indent.ts 0.00% <0.00%> (ø)
packages/codemirror-graphql/src/variables/mode.ts 79.48% <ø> (ø)
packages/graphiql-react/src/editor/whitespace.ts 100.00% <ø> (ø)
packages/graphiql-react/src/utility/debounce.ts 0.00% <0.00%> (ø)
packages/graphiql-react/src/editor/tabs.ts 5.66% <5.66%> (ø)
packages/codemirror-graphql/src/variables/lint.ts 47.61% <66.66%> (+0.63%) ⬆️
packages/codemirror-graphql/src/hint.ts 94.73% <100.00%> (ø)
... and 100 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5827977...35581eb. Read the comment docs.

@acao acao added the lsp-server graphql-language-service-server label Aug 1, 2022
@Foo-x Foo-x marked this pull request as draft August 1, 2022 13:19
@Foo-x
Copy link
Contributor Author

Foo-x commented Aug 1, 2022

Draft until #2616 is merged.

@acao
Copy link
Member

acao commented Aug 11, 2022

oh yes, this makes a lot of sense. let me change the base to your PR

@acao acao changed the base branch from main to revert-2612-revert-multi-root August 11, 2022 18:08
@acao acao force-pushed the revert-2612-revert-multi-root branch from 73d6f68 to 9306f66 Compare October 8, 2022 23:09
@acao acao deleted the branch graphql:revert-2612-revert-multi-root October 10, 2022 19:34
@acao acao closed this Oct 10, 2022
@Foo-x Foo-x deleted the extract-create-diagnostics branch November 10, 2022 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lsp-server graphql-language-service-server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants