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

feat: expose gql client to nitro #258

Merged
merged 1 commit into from
Nov 25, 2022
Merged

feat: expose gql client to nitro #258

merged 1 commit into from
Nov 25, 2022

Conversation

Diizzayy
Copy link
Owner

This PR exposes gql clients to nitro, hence allowing for Gql Functions to be utilized in server routes and server middleware.

Closes #244

@Diizzayy Diizzayy added the enhancement New feature or request label Nov 21, 2022
@Diizzayy Diizzayy self-assigned this Nov 21, 2022
@codecov
Copy link

codecov bot commented Nov 21, 2022

Codecov Report

Merging #258 (f300905) into main (237110a) will increase coverage by 1.13%.
The diff coverage is 94.04%.

@@            Coverage Diff             @@
##             main     #258      +/-   ##
==========================================
+ Coverage   91.50%   92.63%   +1.13%     
==========================================
  Files           6        6              
  Lines         553      611      +58     
  Branches      104      119      +15     
==========================================
+ Hits          506      566      +60     
+ Misses         46       44       -2     
  Partials        1        1              
Impacted Files Coverage Δ
src/module.ts 88.00% <91.93%> (+2.23%) ⬆️
src/context.ts 98.48% <100.00%> (ø)
src/utils.ts 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Diizzayy Diizzayy merged commit c7fe0e0 into main Nov 25, 2022
@Diizzayy Diizzayy deleted the feat/expose-nitro branch November 25, 2022 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Exposing the client in server routes
1 participant