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

Service_call_qos #161

Merged
merged 8 commits into from
Aug 22, 2024
Merged

Service_call_qos #161

merged 8 commits into from
Aug 22, 2024

Conversation

Nikhil-Singhal-06
Copy link
Member

Description

#156

Pull request checklist

  • Issue created that explains the change and why it's needed
  • Tests are part of the PR (for bug fixes / features)
  • Docs reviewed and added / updated if needed (for bug fixes / features)
  • Format and Lint checks (make check) pass locally
  • PR applies Apache 2.0 License to all code files

Pull request type

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation changes
  • Other (please describe)

Current behavior

New behavior

How to test

@Nikhil-Singhal-06 Nikhil-Singhal-06 changed the title add param to service_call and fix pylint warnings Service_call_qos Aug 20, 2024
Copy link

github-actions bot commented Aug 20, 2024

Test Results

275 tests   267 ✅  36m 29s ⏱️
 16 suites    8 💤
 16 files      0 ❌

Results for commit 245222d.

♻️ This comment has been updated with latest results.

@fred-labs fred-labs merged commit af67a52 into main Aug 22, 2024
28 checks passed
@fred-labs fred-labs deleted the feature/service_call_qos branch August 22, 2024 11:03
fred-labs pushed a commit that referenced this pull request Aug 22, 2024
fmirus pushed a commit that referenced this pull request Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants