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

Cumulative Fixes and Improvements #2255

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Cumulative Fixes and Improvements #2255

wants to merge 2 commits into from

Conversation

allentiak
Copy link
Member

@allentiak allentiak commented Dec 13, 2024

No description provided.

Copy link

codecov bot commented Dec 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.07%. Comparing base (45f8df4) to head (61c4925).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2255   +/-   ##
=======================================
  Coverage   95.07%   95.07%           
=======================================
  Files         345      345           
  Lines       20745    20745           
  Branches      483      483           
=======================================
  Hits        19723    19723           
  Misses        539      539           
  Partials      483      483           

@allentiak allentiak changed the title Make all Throwable parameters use the same name (preparation for #2126) Improve naming (preparation for #2126) Dec 13, 2024
@allentiak allentiak marked this pull request as draft December 13, 2024 17:03
* `generate-error-payload` (and not just `error`) is what the function
actually does
* Whereas the names `e` and `ex` are quite similar, they can still
generate some confusion.
@allentiak allentiak marked this pull request as ready for review December 14, 2024 13:19
@allentiak allentiak changed the title Improve naming (preparation for #2126) Cumulative Fixes and Improvements Dec 14, 2024
@allentiak allentiak added the test-fix Fixed a test. label Dec 14, 2024
The body should contain a JSON payload with the OperationOutcome, not
the OperationOutcome itself.
We use JSON because it is our default.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant