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

Add operation id #691

Merged
merged 1 commit into from
Sep 9, 2024
Merged

Add operation id #691

merged 1 commit into from
Sep 9, 2024

Conversation

YoshihitoAso
Copy link
Member

close #646

@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 6, 2024

Coverage

Coverage Report •
FileStmtsMissCoverMissing
issuer
   account.py181298%501, 503
   bond.py11201199%716, 1128, 2123, 2335, 2499, 2796, 2804, 3744–3747
   common.py49197%48
   file.py820100% 
   ledger.py282398%916–917, 958
   notification.py450100% 
   position.py185199%358
   share.py11021199%683, 1095, 2051, 2263, 2427, 2727, 2735, 3674–3677
   token_holders.py200199%593
misc
   e2e_messaging.py2020100% 
   settlement_agent.py164298%218, 318
TOTAL12958113791% 

Tests Skipped Failures Errors Time
1896 0 💤 0 ❌ 0 🔥 33m 37s ⏱️

@YoshihitoAso YoshihitoAso marked this pull request as ready for review September 6, 2024 07:13
purplesmoke05
purplesmoke05 approved these changes Sep 9, 2024
View reviewed changes
Copy link
Member

@purplesmoke05 purplesmoke05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@purplesmoke05 purplesmoke05 merged commit c509f53 into dev-24.9 Sep 9, 2024
8 checks passed
@purplesmoke05 purplesmoke05 deleted the feature/#646 branch September 9, 2024 10:19
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@purplesmoke05 purplesmoke05 purplesmoke05 approved these changes

Assignees

@YoshihitoAso YoshihitoAso

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEATURE] Define operationId to each API path
2 participants
@YoshihitoAso @purplesmoke05