Skip to content

feat(core): improve roundtripper error message #1846

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
Merged
merged 2 commits into from
Apr 9, 2025
Merged

feat(core): improve roundtripper error message #1846

merged 2 commits into from
Apr 9, 2025

Conversation

marceljk
Copy link
Contributor

@marceljk marceljk commented Apr 9, 2025

Description

relates to #731 and STACKITSDK-53

Checklist

  • Issue was linked above
  • No generated code was adjusted manually (check comments in file header)
  • Changelogs
    • Changelog in the root directory was adjusted (see here)
    • Changelog(s) of the service(s) were adjusted (see e.g. here)
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Unit tests got implemented or updated
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@marceljk marceljk force-pushed the feat/improve-roundtripper-error-messages branch from 563a557 to 4f6603e Compare April 9, 2025 13:55
bahkauv70
bahkauv70 reviewed Apr 9, 2025
View reviewed changes
@marceljk marceljk merged commit 2e89a06 into main Apr 9, 2025
8 checks passed
@marceljk marceljk deleted the feat/improve-roundtripper-error-messages branch April 9, 2025 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bahkauv70 bahkauv70 bahkauv70 approved these changes

@Fyusel Fyusel Awaiting requested review from Fyusel

@rubenhoenle rubenhoenle Awaiting requested review from rubenhoenle

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@marceljk @bahkauv70