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

Use native types #177

Merged
merged 54 commits into from
Mar 5, 2023
Merged

Use native types #177

merged 54 commits into from
Mar 5, 2023

Conversation

kitloong
Copy link
Collaborator

@kitloong kitloong commented Mar 3, 2023

Fix #170, #143, #120

@kitloong kitloong requested a review from kevincobain2000 March 3, 2023 09:01
@kitloong kitloong marked this pull request as ready for review March 3, 2023 09:01
@codecov-commenter
Copy link

codecov-commenter commented Mar 3, 2023

Codecov Report

Merging #177 (717dfdf) into master (b0b6771) will increase coverage by 51.32%.
The diff coverage is 92.51%.

@@              Coverage Diff              @@
##             master     #177       +/-   ##
=============================================
+ Coverage     41.38%   92.71%   +51.32%     
+ Complexity      179      144       -35     
=============================================
  Files             6        6               
  Lines           505      453       -52     
=============================================
+ Hits            209      420      +211     
+ Misses          296       33      -263     
Impacted Files Coverage Δ
src/LaravelRequestDocsServiceProvider.php 95.23% <50.00%> (-4.77%) ⬇️
src/LaravelRequestDocsMiddleware.php 61.84% <61.11%> (+61.84%) ⬆️
src/LaravelRequestDocs.php 99.10% <98.66%> (+26.58%) ⬆️
src/Controllers/LaravelRequestDocsController.php 100.00% <100.00%> (+100.00%) ⬆️
src/LaravelRequestDocsToOpenApi.php 98.86% <100.00%> (+98.86%) ⬆️
src/NotFoundWhenProduction.php 100.00% <100.00%> (+100.00%) ⬆️
... and 1 more

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

@kitloong kitloong merged commit 63511df into master Mar 5, 2023
@kitloong kitloong deleted the feature/types branch March 5, 2023 15:15
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.

Add a wrapper to wrap the response when request from LRD UI.
3 participants