Skip to content

Don't dump records into temporary file - #127

Merged
exoego merged 1 commit into
exoego:masterfrom
andyundso:remove-tmp-records
Aug 15, 2023
Merged

Don't dump records into temporary file#127
exoego merged 1 commit into
exoego:masterfrom
andyundso:remove-tmp-records

Conversation

@andyundso

Copy link
Copy Markdown
Contributor

I browsed the source code the other day and found this statement. As it is referenced nowhere else in the code, I assume it was used for debugging purposes when implementing the Minitest support and is therefore no longer necessary.

I browsed the source code the other day and found this statement. As it is referenced nowhere else in the code, I assume it was used for debugging purposes when implementing the Minitest support and is therefore no longer necessary.
@codecov

codecov Bot commented Jul 26, 2023

Copy link
Copy Markdown

Codecov Report

Merging #127 (5baea92) into master (0c92c20) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #127      +/-   ##
==========================================
- Coverage   96.63%   96.62%   -0.01%     
==========================================
  Files          14       14              
  Lines         416      415       -1     
==========================================
- Hits          402      401       -1     
  Misses         14       14              
Files Changed Coverage Δ
lib/rspec/openapi/result_recorder.rb 88.00% <ø> (-0.47%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@exoego

exoego commented Aug 15, 2023

Copy link
Copy Markdown
Owner

@exoego
exoego merged commit ada8ce3 into exoego:master Aug 15, 2023
@andyundso
andyundso deleted the remove-tmp-records branch October 16, 2023 04:37
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