Diizzayy / nuxt-graphql-client

⚡️ Minimal GraphQL Client + Code Generation for Nuxt3
https://nuxt-graphql-client.web.app
MIT License
366 stars 44 forks source link

perf: prevent superfluous code generation #235

Closed Diizzayy closed 2 years ago

Diizzayy commented 2 years ago

This PR foregoes code generation during build and when no documents have been matched to the configured client(s)

codecov[bot] commented 2 years ago

Codecov Report

Merging #235 (bdec75b) into main (397b675) will decrease coverage by 13.99%. The diff coverage is 72.36%.

:exclamation: Current head bdec75b differs from pull request most recent head 426a173. Consider uploading reports for the commit 426a173 to get more accurate results

@@             Coverage Diff             @@
##             main     #235       +/-   ##
===========================================
- Coverage   93.22%   79.22%   -14.00%     
===========================================
  Files           5        5               
  Lines         546      544        -2     
  Branches      109       82       -27     
===========================================
- Hits          509      431       -78     
- Misses         37      113       +76     
Impacted Files Coverage Δ
src/generate.ts 35.89% <56.25%> (-50.77%) :arrow_down:
src/module.ts 88.17% <76.47%> (-3.05%) :arrow_down:
src/context.ts 78.46% <76.92%> (-20.80%) :arrow_down:

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