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.

Sign up for GitHub

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

Jump to bottom

fix: Pass the operationId option correctly to @hey-api/openapi-ts #151

Open
wants to merge 1 commit into
base: v1
Choose a base branch
from
Open

fix: Pass the operationId option correctly to @hey-api/openapi-ts #151

wants to merge 1 commit into from

Conversation

7nohe
Copy link
Owner

@7nohe 7nohe commented Sep 24, 2024

fix #133

@github-actions GitHub Actions
Copy link

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 99.02% (🎯 95%) 2034 / 2054
🟢 Statements 99.02% (🎯 95%) 2034 / 2054
🟢 Functions 100% (🎯 95%) 41 / 41
🟢 Branches 93.43% (🎯 85%) 185 / 198
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
src/generate.mts 100% 50% 100% 100%
Generated in workflow #334

@7nohe 7nohe self-assigned this Sep 24, 2024
@7nohe
Copy link
Owner Author

7nohe commented Sep 24, 2024

npm publish

@7nohe 7nohe mentioned this pull request Sep 24, 2024
Closed
@7nohe 7nohe requested a review from seriouslag September 25, 2024 00:54
@7nohe 7nohe changed the base branch from main to v1 October 14, 2024 05:56
@7nohe 7nohe added the v1 label Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seriouslag seriouslag seriouslag approved these changes

Assignees

@7nohe 7nohe

Labels
v1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

"--operationId" still includes service name
2 participants
@7nohe @seriouslag