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

chore(deps): update @hey-api/openapi-ts to 0.42.1 #84

Merged
merged 2 commits into from
Apr 23, 2024

Conversation

seriouslag
Copy link
Collaborator

@seriouslag seriouslag commented Apr 22, 2024

In this PR:

@seriouslag seriouslag requested a review from 7nohe April 22, 2024 02:04
Copy link

github-actions bot commented Apr 22, 2024

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 98.96% (🎯 95%) 1524 / 1540
🟢 Statements 98.96% (🎯 95%) 1524 / 1540
🟢 Functions 100% (🎯 95%) 35 / 35
🟢 Branches 94.28% (🎯 85%) 132 / 140
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
src/common.mts 100% 100% 100% 100%
src/constants.mts 100% 100% 100% 100%
src/createImports.mts 96.15% 71.42% 100% 96.15% 22-25, 29
src/createSource.mts 100% 100% 100% 100%
src/createUseMutation.mts 99.61% 93.33% 100% 99.61% 83
src/createUseQuery.mts 100% 100% 100% 100%
src/format.mts 100% 100% 100% 100%
src/generate.mts 100% 100% 100% 100%
src/print.mts 100% 100% 100% 100%
src/service.mts 90.82% 76.19% 100% 90.82% 42-43, 50-51, 57-58, 62-63, 73-74
src/util.mts 100% 100% 100% 100%
Generated in workflow #226

@seriouslag seriouslag force-pushed the deps/update-hey-api-0.42.1 branch 2 times, most recently from 864d6c0 to 82357ea Compare April 22, 2024 03:12
@seriouslag seriouslag force-pushed the deps/update-hey-api-0.42.1 branch 2 times, most recently from 1a33e21 to f70fd02 Compare April 22, 2024 18:40
@seriouslag seriouslag added bug Something isn't working enhancement New feature or request labels Apr 22, 2024
- Update to the latest `@hey-api/openapi-ts` version `0.42.1`
  - fixes: #83,
  - Enables work towards #81
- Fixed a bug where types in mutation hooks were imported from a (non-relative) direct file import.
    - This is the same fix implemented in #61, which was done for query hooks.
    - Expand the match regex to take account of non-alphabetical file paths
      - fixes: #85
- Fixed a bug where JSDocs were not being added to generated hooks.
- Add more unit testing
@seriouslag seriouslag self-assigned this Apr 23, 2024
We now format the generated hooks file to remove unused imports and fix some formatting issues.
@7nohe 7nohe merged commit e360dcd into main Apr 23, 2024
4 checks passed
@7nohe 7nohe deleted the deps/update-hey-api-0.42.1 branch April 23, 2024 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
2 participants